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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include instructions to run SystemInfoTest #301

Merged
merged 1 commit into from Jan 14, 2017

Conversation

ckadner
Copy link
Contributor

@ckadner ckadner commented Jan 14, 2017

It is not very straight forward to run the advertised SystemInfoTest from the command line unless one is eager to download a few jars and add them to the classpath (SLF4J, JNI, JSON).

For people familiar with Git and Maven, running a mvn command to execute SystemInfoTest will require less effort ...and if included in the README it comes down to a simple copy and paste 馃槃

mvn test-compile -pl oshi-core -q exec:java \
  -Dexec.mainClass="oshi.SystemInfoTest" \
  -Dexec.classpathScope="test"

@codecov-io
Copy link

codecov-io commented Jan 14, 2017

Current coverage is 84.62% (diff: 100%)

Merging #301 into master will not change coverage

@@             master       #301   diff @@
==========================================
  Files            23         23          
  Lines          1080       1080          
  Methods           0          0          
  Messages          0          0          
  Branches        195        195          
==========================================
  Hits            914        914          
  Misses           72         72          
  Partials         94         94          

Powered by Codecov. Last update 4e654e6...1ac11a3

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.222% when pulling 1ac11a3 on ckadner:how_to_run_SystemInfoTest_with_mvn into 4e654e6 on oshi:master.

@dbwiddis dbwiddis merged commit b90292c into oshi:master Jan 14, 2017
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.

None yet

4 participants