Github user squito commented on the pull request:

    https://github.com/apache/spark/pull/10107#issuecomment-161788283
  
    you should mimic the folder structure of the main class, just w/ "test" 
instead of main".  So for 
"core/src/main/scala/org/apache/spark/status/api/v1/AllStagesResource.scala", 
you would make 
"core/src/test/scala/org/apache/spark/status/api/v1/AllStagesResourceSuite.scala".
    
    I'm not suggesting you add another expecation file at all.  
HistoryServerSuite is a little unusual in that the tests reference an external 
resource for what the results should be. You can write a much more 
straightforward test, it will just call the relevant method in 
`AllStagesResource`, and do some checks on the result.  for a really basic 
example consider ByteArrayChunkOutputStreamSuite.
    
    Hopefully that helps, I can also write a skeleton if you are stuck


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to