Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test for classpath with wildcards resetting environment #5760

Merged
merged 1 commit into from
Jul 30, 2017

Conversation

jpallas
Copy link
Contributor

@jpallas jpallas commented Jul 28, 2017

This tests that a wildcard classpath add will only reset the environment once (#5701). The change also decouples this test suite from the structure of the doc directory. Dummy jar files are used, since the tests don't depend on the files being valid jars.

@scottdraves scottdraves merged commit 95bf86f into twosigma:master Jul 30, 2017
@scottdraves
Copy link
Contributor

awesome, thank you.

michalgce added a commit that referenced this pull request Aug 4, 2017
* origin/master: (25 commits)
  #5695 fix legend position in png/svg export (#5785)
  add e2e tests for Kotlin notebook (#5786)
  #5675 Problems with EasyForm Python API  (#5790)
  #5779 Change the interrupt msg. (#5789)
  jitpack, npm and pypi badges added (#5791)
  cancelExecution when interupt action (#5776)
  #5685 set table font to Lato (#5771)
  #5758 Allow extra whitespaces for magic commands. (#5773)
  add tests for javaTutorial.ipynb notebook (#5774)
  fix indentation
  #5658: no tooltip as default (#5775)
  jarek/5766: display tablesaw as TableDisplay (#5772)
  #5097 logo icons for beakerx languages (#5769)
  #5070 fix the adjustRedraw error of tableDisplay (#5750)
  #5147: add jvm-repr (#5764)
  close #5756 by linking to doc from readme
  add tests for ClojureTutorial (#5763)
  #5178 fix heatmap width too low (#5762)
  Add test for classpath with wildcards resetting environment multiple times (#5760)
  #5746 Skip runtime version check for Kotlin. (#5759)
  ...
@jpallas jpallas deleted the classpath-wildcard-test branch September 11, 2017 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants