test: Isolated testing and SentryOptions tests
#80
Conversation
limbonaut
changed the title
SentryOptions tests
| @@ -2,6 +2,7 @@ | |||
| project/addons | |||
| project/export_presets.cfg | |||
| project/.vscode | |||
| project/reports | |||
There was a problem hiding this comment.
For information, reports are created when unit tests are executed in the editor or from CLI.
| @@ -16,6 +16,9 @@ func _configure(options: SentryOptions) -> void: | |||
| options.before_send = _before_send | |||
| options.on_crash = _on_crash | |||
|
|
|||
| # Unit testing hooks (if you're exploring the demo project, pretend the following line doesn't exist). | |||
| TestingConfiguration.configure_options(options) | |||
There was a problem hiding this comment.
Allowing unit tests to hook into our configuration script. I call such tests isolated, as they need to be executed separately from other similar tests. There is a new script that automates running them in a batch.
There was a problem hiding this comment.
TODO: This would break if the demo project were shipped without unit test files, which is how we currently do it.
scripts/run-isolated-tests.sh
Outdated
There was a problem hiding this comment.
One advantage of writing the script in pwsh is that we can run these on Windows/macOS/Linux
There was a problem hiding this comment.
Addressed in 6aefa3c
limbonaut
force-pushed
the
test/isolated-and-options-tests
branch
from
a4fad5f to
5f00372
Compare
|
I have to give it another look. Something fishy is going on when I run these tests in a different environment. |
This PR introduces isolated testing and several new testing goodies:
SentryOptions, error logger, and more.Isolated tests are good for testing various SDK options, as we may configure those only once and only at the app start.
Closes #75
#skip-changelog