[I] add unit test coverage for toQueryStrings [texera]

["eugenegujing (via GitHub)" <gi...@apache.org>]

eugenegujing opened a new issue, #6399:
URL: https://github.com/apache/texera/issues/6399

   ### Task Summary
   
   `frontend/src/app/dashboard/type/search-filter-parameters.ts` exports `toQueryStrings`, the pure query-string builder that every dashboard and hub search request flows through: it turns keywords, the four date filters, the `owners`/`ids`/`operators`/`projectIds` filters, pagination, resource type, and sort order into the encoded query string sent to the search endpoints.
   
   It has no spec, and none of its construction logic is pinned anywhere: the search tests in the existing `search.service.spec.ts` call it through `SearchService.search` but only assert the endpoint choice and the `includePublic` flag (which `SearchService` appends itself, outside `toQueryStrings`) against an empty filter, and the file's remaining tests never touch `toQueryStrings`.
   
   ### Task Type
   
   - [ ] Refactor / Cleanup
   - [ ] DevOps / Deployment / CI
   - [x] Testing / QA
   - [ ] Documentation
   - [ ] Performance
   - [ ] Other


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] add unit test coverage for toQueryStrings [texera]

["eugenegujing (via GitHub)" <gi...@apache.org>]

eugenegujing commented on issue #6399:
URL: https://github.com/apache/texera/issues/6399#issuecomment-4963318487

   /take


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] add unit test coverage for toQueryStrings [texera]

["aglinxinyuan (via GitHub)" <gi...@apache.org>]

aglinxinyuan closed issue #6399: add unit test coverage for toQueryStrings
URL: https://github.com/apache/texera/issues/6399


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org