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

New requirement to ensure is exactly one digit to the left of the decimal point for the specifier e or E and fix tests #1042

Merged
merged 12 commits into from
Oct 24, 2024

Conversation

al-niessner
Copy link
Contributor

🗒️ Summary

Fix field regular expresion

⚙️ Test Data and/or Report

All automated unit/regression tests pass

♻️ Related Issues

Closes #992

@al-niessner al-niessner requested a review from a team as a code owner October 22, 2024 20:15
@al-niessner al-niessner self-assigned this Oct 22, 2024
Copy link
Member

@nutjob4life nutjob4life left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@al-niessner am I running these tests wrong?

$ echo $JAVA_HOME
/Library/Java/JavaVirtualMachines/jdk-17.0.1.jdk/Contents/Home
$ mvn --version
Java version: 17.0.1, vendor: Oracle Corporation, runtime: /Library/Java/JavaVirtualMachines/jdk-17.0.1.jdk/Contents/Home
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "15.0.1", arch: "x86_64", family: "mac"
$ git status
On branch issue_992
Your branch is up to date with 'origin/issue_992'.

nothing to commit, working tree clean
$ mvn clean test
[INFO] Scanning for projects...
[INFO] Inspecting build with total of 1 modules...
[INFO] Installing Nexus Staging features:
[INFO]   ... total of 1 executions of maven-deploy-plugin replaced with nexus-staging-maven-plugin
[INFO] 
[INFO] -----------------------< gov.nasa.pds:validate >------------------------
[INFO] Building gov.nasa.pds:validate 3.6.0-SNAPSHOT
[INFO]   from pom.xml
[INFO] --------------------------------[ jar ]---------------------------------
[INFO] 
[INFO] --- clean:3.2.0:clean (default-clean) @ validate ---
[INFO] Deleting /Users/kelly/Documents/Clients/JPL/PDS/Development/nasa-pds/validate/target
[INFO] 
[INFO] --- buildnumber:3.2.1:create (default) @ validate ---
[INFO] Storing buildNumber: 2024-10-22 17:00:52 at timestamp: 1729634452655
[INFO] Executing: /bin/sh -c cd '/Users/kelly/Documents/Clients/JPL/PDS/Development/nasa-pds/validate' && 'git' 'symbolic-ref' 'HEAD'
[INFO] Working directory: /Users/kelly/Documents/Clients/JPL/PDS/Development/nasa-pds/validate
[INFO] Storing scmBranch: issue_992

[ERROR] Tests run: 270, Failures: 60, Errors: 0, Skipped: 23
[INFO] 
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  10:29 min
[INFO] Finished at: 2024-10-22T17:11:17-05:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:3.3.0:test (default-test) on project validate: There are test failures.
[ERROR] 
[ERROR] Please refer to /Users/kelly/Documents/Clients/JPL/PDS/Development/nasa-pds/validate/target/surefire-reports for the individual test results.
[ERROR] Please refer to dump files (if any exist) [date].dump, [date]-jvmRun[N].dump and [date].dumpstream.
[ERROR] -> [Help 1]

@al-niessner
Copy link
Contributor Author

al-niessner commented Oct 22, 2024 via email

@nutjob4life
Copy link
Member

@al-niessner no worries, no rush! I'll stand by for now. (Well, not really … I'll sit by because I'm already on my chair 😆)

@al-niessner al-niessner changed the title only number before . only one number before . Oct 23, 2024
@al-niessner
Copy link
Contributor Author

@jordanpadams @nutjob4life @tloubrieu-jpl

Can I please get a quick review and merge so that I can repair some of the other branches. Thanks.

@nutjob4life
Copy link
Member

You got it @al-niessner! I just pulled and am running mvn test.

@nutjob4life nutjob4life self-requested a review October 24, 2024 17:25
Copy link
Member

@nutjob4life nutjob4life left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@al-niessner good news!

[INFO] Tests run: 233, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 511.2 s -- in cucumber.CucumberTest
[INFO] 
[INFO] Results:
[INFO] 
[WARNING] Tests run: 264, Failures: 0, Errors: 0, Skipped: 23
[INFO] 
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  10:07 min
[INFO] Finished at: 2024-10-24T12:24:34-05:00
[INFO] ------------------------------------------------------------------------

Let's merge this already! 👍 🎉

@jordanpadams jordanpadams changed the title only one number before . New requirement to ensure is exactly one digit to the left of the decimal point for the specifier e or E and fix tests Oct 24, 2024
@jordanpadams jordanpadams merged commit 3dde814 into main Oct 24, 2024
3 checks passed
@jordanpadams jordanpadams deleted the issue_992 branch October 24, 2024 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants