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

fix: make android mocha tests run again #14158

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

m1ga
Copy link
Contributor

@m1ga m1ga commented Dec 18, 2024

I was wondering why the Android mocha tests weren't running anymore. Turns out: the generated tiapp.xml was invalid. There was a missing > and it had two <application> (we have a default one in the template now) nodes.

Fixing some tests:

  • tls 1.3 url was not working. Using a different one
  • redirect url was blocked, using a different one
  • one test was throwing a "getText() of null" error for the label so I've added a null check
  • headerTitle/footerTitle is not working (even with 12.5.0) so disabling that for the moment - Fixed in fix(android): fix global header/footer title in ListView #14160
  • added another density image for the textfield padding test

Interesting point:
tableView.deleteRow(tableView.sections[0].rows[2]); is not working in the emulator but on a device it is working fine 🤷 That's one test that is still failing but couldn't fix it yet.

@m1ga
Copy link
Contributor Author

m1ga commented Jan 12, 2025

@prashantsaini1 maybe you can check this one too. Then the testsuite will run again (npm run tests). One small null check in the SDK, the rest is just JS fixes.

Just a note: there are still around 10 tests that fail in the emulator, but the outher thousand of tests run fine 😄

@prashantsaini1
Copy link
Contributor

This one's fine as I already noticed at other places that getNativeView is being used without null checks, whereas it should be guarded.

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.

2 participants