Skip to content

Conversation

@jrmartin
Copy link
Collaborator

@jrmartin jrmartin commented Sep 5, 2025

This pull request modernizes the SDS Viewer’s test and build infrastructure, refactors test utility modules for consistency, and introduces a new debug mode for dataset upload and visibility. The most significant changes are grouped below:

Continuous Integration and Node Environment Updates:

  • Updated the GitHub Actions workflow (.github/workflows/actions.yml) to use Ubuntu 22.04 and Node.js 18.x, modernizing the CI environment and improving compatibility and security. Also, the workflow now triggers on both master and development branches. [1] [2]
  • Upgraded testing dependencies in package.json to the latest major versions of jest, jest-puppeteer, jest-image-snapshot, and puppeteer, ensuring improved reliability and future compatibility. The E2E test script was also updated for clarity.

Test Utilities Refactor:

  • Converted selectors.js and time_constants.js from ES module exports to CommonJS (module.exports), aligning with the test runner’s requirements and simplifying imports. [1] [2]
  • Updated all test imports in e2e.test.js to use CommonJS require syntax, ensuring consistency and compatibility with the refactored utility modules. Also, set the default development URL to http://localhost:3000/ for local testing.

Debug Mode and Dataset Upload Improvements:

  • Added a debug mode to the SDS Viewer, documented in README.md, allowing users to upload datasets from local disk and view all datasets regardless of status by using the ?debug=true URL parameter.
  • Refactored DatasetsListDialog and related components to support debug mode and enable/disable the upload tab based on props, including logic to handle file uploads and dataset filtering. [1] [2] [3] [4] [5]
  • Updated the main app component to pass new enableUpload and debug props, and removed legacy upload dialog logic. [1] [2]

Miscellaneous Fixes and Improvements:

  • Fixed a potential runtime error in dataset matching by adding a null check for doi in App.js.
  • Corrected visibility logic in mainLayout.js to properly identify visible child nodes.
  • Minor UI and event handling improvements in dialogs and components. [1] [2] [3] [4]

These changes collectively improve the developer experience, test reliability, and add a powerful debug mode for advanced dataset handling.

jrmartin and others added 30 commits May 20, 2025 06:59
…th-tabs

#247 - update jest puppeteer, increase allows upload size
…th-tabs

#247 - Increase file size allowed on file upload, save debug more on local storage
Fix typo with name

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
jrmartin and others added 20 commits August 21, 2025 09:32
…el-for-missing-properties

feat: include missing metadata properties
…or-folder-hierarchy

fix: show folder names without full path in tree
…-splinter.getgraph

feat: show primary and derivative folders
…or-folder-hierarchy

fix: simplify folder linking and improve node attribute handling in Splinter utility
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ey-md9vwd

feat: configure folder icon colors for graph nodes
@jrmartin jrmartin changed the title Development - SDS 3.0 Introduction Development - SDS 3.0 Sep 5, 2025
@jrmartin jrmartin marked this pull request as ready for review September 5, 2025 02:53
Copilot AI and others added 8 commits October 30, 2025 17:58
Co-authored-by: jrmartin <4562825+jrmartin@users.noreply.github.com>
Co-authored-by: jrmartin <4562825+jrmartin@users.noreply.github.com>
Co-authored-by: jrmartin <4562825+jrmartin@users.noreply.github.com>
Co-authored-by: jrmartin <4562825+jrmartin@users.noreply.github.com>
Modified duplicate detection logic to only skip folder creation when the
folder is a sparc node AND the parent is NOT that sparc node. This allows
primary/derivative folders to be displayed under their sparc node
representation while still preventing duplicates elsewhere.

For example:
- sam-l-seg-c1/primary folder now shows under sam-l-seg-c1 Sample node
- But sam-l-seg-c1 does NOT appear as duplicate under sam-l/primary

Co-authored-by: jrmartin <4562825+jrmartin@users.noreply.github.com>
Fix duplicate Collection nodes for folders already represented as sparc: entities
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.

3 participants