Skip to content

Andres feature#134

Open
AndresDoesCode wants to merge 3 commits into
urban-toolkit:mainfrom
AndresDoesCode:Andres-feature
Open

Andres feature#134
AndresDoesCode wants to merge 3 commits into
urban-toolkit:mainfrom
AndresDoesCode:Andres-feature

Conversation

@AndresDoesCode

Copy link
Copy Markdown

Describe your changes

I added more test cases covering the Backend and Sandbox

Sandbox Tests

  • test_exec_js_receives_input_from_duckdb
    JS code receives Python-DuckDB-stored input via the arg parameter
  • test_exec_js_float_stored_correclty_in_duckdb
    Floats can be inserted and accsess from duckDB without any side effects
  • test_exec_js_string_stored_correclty_in_duckdb
    Strings are stored and accsessed gracefully with DuckDB
  • test_interactions_between_Python_and_JS_node_types
    Tests whether data will flow nicely from Python to JS nodes
  • test_empty_returns
    null and undefined return types are handled gracefully
  • test_artifacts_and_session_ids
    This tests the isolation of sessions. Can one session grab information from another?

Backend Tests

  • test_load_shared_project
  • test_rename_project
  • test_put_delete_duplicate_for_missing_project
  • test_signin_google_returns_200 (test_routes)
  • test_invalid_token_returns_401 (test_routes)
  • test_signin_google_returns_200 (test_services)
  • test_invalid_token_returns_401 (test_services)

Type of change (Check all that apply)

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update
  • Other:

Parts of Curio impacted by this PR:

  • Frontend
  • Backend
  • Sandbox

Testing

  • Executed ./scripts/test.sh and it passed all tests.

Checklist (Check all that apply)

  • I have manually loaded each .json test from the tests/ folder into Curio, ran all the nodes one by one, and checked that they run without errors and give the expected results
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

I added more test cases to the sandbox
I finished writing test cases for the backend
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.

1 participant