Merge pull request #700 from NASA-PDS/issue_406

Add API support for checking if a search field exists in a result (e.g. not null)

Author: jordanpadams

CHANGELOG.md

@@ -2,7 +2,7 @@
 
 ## [1.6.0](https://github.com/NASA-PDS/registry-api/tree/1.6.0) (2025-10-14)
 
-[Full Changelog](https://github.com/NASA-PDS/registry-api/compare/v1.6.2...1.6.0)
+[Full Changelog](https://github.com/NASA-PDS/registry-api/compare/1.6.0...v1.6.2)
 
 **Defects:**
 

CONTRIBUTING.md

@@ -63,8 +63,24 @@ There are a few steps required to prepare for merging code back into the main br
 
 1. Create a pull request if have not done this already.
 1. Address all automated messages.
-1. Run all regression checks to make sure changes have re-introduced already fixed bugs.
+1. **Add required integration tests** (see [Integration Testing Requirements](#integration-testing-requirements) below).
+1. Run all regression checks to make sure changes have not re-introduced already fixed bugs.
 1. Move from draft to ready for review if in draft mode.
 1. Request review.
 
+## Integration Testing Requirements
+
+**IMPORTANT**: Each new feature, requirement, or bug fix must include at least one integration test added to the Postman collection.
+
+Integration tests are maintained in the [`registry` repository](https://github.com/NASA-PDS/registry) and must be updated as part of your contribution. For detailed instructions on creating and submitting integration tests, see:
+
+**[Integration Testing Guide](https://nasa-pds.github.io/registry/developer/integration-testing.html)**
+
+The guide covers:
+- When tests are required
+- Step-by-step process for adding tests to Postman
+- TestRail integration (for internal developers)
+- Running and validating tests locally
+- Submitting test updates via pull request
+
 

README.md

@@ -113,23 +113,19 @@ The integration tests will be automatically applied. Check the results, update/c
 
 ## Tests
 
-**Important note:** As a developer you are asked to complete the postman test suite according to the new feature you are developing. Do a pull request in the `registry` project to submit the updates.
+### Testing Requirements
 
-Integration test are maintained in postman.
+**IMPORTANT:** As a developer, you are **required** to add integration tests to the Postman test suite for:
+- Each new feature or requirement
+- Each bug fix
+- Any changes to existing API behavior
 
-### Edit/Run of the integration tests in postman GUI
+### Integration Testing Guide
 
-Install the postman desktop, from https://www.postman.com/downloads/
+Integration tests are maintained in the `registry` repository as Postman collections. For complete instructions on creating, running, and submitting integration tests, see:
 
-Download and open the test suite found in https://github.com/NASA-PDS/registry/tree/main/docker/postman
+**[Integration Testing Guide](https://nasa-pds.github.io/registry/developer/integration-testing.html)**
 
-### Run the integration tests in command line
-
-In the `registry` project.
-
-Launch the test in command line:
-
-    npm install newman
-    newman run docker/postman/postman_collection.json --env-var baseUrl=http://localhost:8080
+All test updates must be submitted as pull requests to the [`registry` repository](https://github.com/NASA-PDS/registry).
 
 

docker/Dockerfile.local

@@ -46,12 +46,3 @@ For example on AWS, with OpenSearch serverless as a back-end:
 
     SPRING_BOOT_APP_ARGS=--openSearch.host=<your opensearch serverless URL> --openSearch.CCSEnabled=true --openSearch.username="" --openSearch.disciplineNodes=atm-delta,en-delta --registry.service.version=1.5.0-SNAPSHOT
     SERVER_PORT=80
-
-
-
-
-
-
-## 📍 Dockerfile.local
-
-You can ignore `Dockerfile.local` unless you're @al-niessner.

docker/README.md

@@ -1,8 +1,9 @@
 grammar Search;
 
 query : queryTerm EOF ; 
-queryTerm : comparison | likeComparison | group ;
+queryTerm : comparison | likeComparison | existence | group ;
 group : NOT? LPAREN expression RPAREN ;
+existence : ( FIELD | STRINGVAL ) EXISTS ;
 expression : andStatement | orStatement | queryTerm ;
 andStatement : queryTerm (AND queryTerm)+ ;
 orStatement : queryTerm (OR queryTerm)+ ;
@@ -19,6 +20,7 @@ GE : G E ;
 LT : L T ;
 LE : L E ;
 
+EXISTS: E X I S T S;
 LIKE: L I K E;
 
 LPAREN : '(' ;

lexer/src/main/antlr4/gov/nasa/pds/api/registry/lexer/Search.g4

@@ -10,6 +10,7 @@
 import gov.nasa.pds.api.registry.lexer.SearchParser.ExpressionContext;
 import gov.nasa.pds.api.registry.lexer.SearchParser.GroupContext;
 import gov.nasa.pds.api.registry.lexer.SearchParser.LikeComparisonContext;
+import gov.nasa.pds.api.registry.lexer.SearchParser.ExistenceContext;
 import gov.nasa.pds.api.registry.lexer.SearchParser.OperatorContext;
 import gov.nasa.pds.api.registry.lexer.SearchParser.OrStatementContext;
 import gov.nasa.pds.api.registry.lexer.SearchParser.QueryContext;
@@ -23,73 +24,73 @@ public class MockedListener implements ParseTreeListener, SearchListener {
 
   @Override
   public void enterQuery(QueryContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void exitQuery(QueryContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void enterQueryTerm(QueryTermContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void exitQueryTerm(QueryTermContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void enterGroup(GroupContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void exitGroup(GroupContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void enterExpression(ExpressionContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void exitExpression(ExpressionContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void enterAndStatement(AndStatementContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void exitAndStatement(AndStatementContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void enterOrStatement(OrStatementContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void exitOrStatement(OrStatementContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
@@ -102,7 +103,7 @@ public void enterComparison(ComparisonContext ctx) {
 
   @Override
   public void exitComparison(ComparisonContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
@@ -118,44 +119,57 @@ public void enterLikeComparison(LikeComparisonContext ctx) {
 
   @Override
   public void exitLikeComparison(LikeComparisonContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void enterOperator(OperatorContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void exitOperator(OperatorContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void visitTerminal(TerminalNode node) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void visitErrorNode(ErrorNode node) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void enterEveryRule(ParserRuleContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
   @Override
   public void exitEveryRule(ParserRuleContext ctx) {
-    // TODO Auto-generated method stub
+    // Nothing useful to do in this mocked version
 
   }
 
+
+  @Override
+  public void enterExistence(ExistenceContext ctx) {
+    // Nothing useful to do in this mocked version
+
+  }
+
+  @Override
+  public void exitExistence(ExistenceContext ctx) {
+    this.field = ctx.FIELD();
+    this.strval = ctx.STRINGVAL();
+  }
+
 }