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

add the last bulk batch of Python tests to JS #2629

Merged
merged 2 commits into from
Jun 2, 2023

Conversation

mmomtchev
Copy link
Contributor

@mmomtchev mmomtchev commented Jun 1, 2023

This is the last bulk batch of Python run tests translated to JS

There are about 100 remaining tests most of which are blocked by one of these:

  • directors do not work in JS
  • nested struct/unions do not work in JS
  • overloading between integers and floats do not work in JS (there is a separate PR [JS] Use CanCastAsInteger in JSC and V8 #2624 with this)
  • the std containers are less evolved and there is no std::list
  • there is no automatic implicit conversion between language builtin containers and std containers - this is a priority
  • there are no built-in complex numbers in JS
  • there is no Doxygen support
  • lots of cases related to renaming, especially renaming constructors
  • some tests related to reference counting - alas with JS engines it is impossible to force the GC to run which is always a problem when testing mixed JS/C++ code

@ojwb ojwb added the Javascript label Jun 2, 2023
@ojwb ojwb merged commit d05e270 into swig:master Jun 2, 2023
104 checks passed
@mmomtchev mmomtchev deleted the js-run-tests-last-batch branch June 2, 2023 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants