-
Notifications
You must be signed in to change notification settings - Fork 35
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
Client tests don't work with changes in Mojo 24.3 #34
Comments
Here is the full error that throws when trying to run
|
I updated
launched the lightbug server in a separate terminal, then ran the tests and it got past the compilation errors. But it did fail with what seems to be a string termination issue. And also maybe using the wrong service (content length doesn't match)
|
Oops, @thatstoasty I think as a server need to run |
Interesting, are you passing host and port as two separate args or a tuple? |
This is if I copy from your previous message, two args, like this: |
Solved now after #61 |
Since Mojo 24.3
main()
functions inside packages are no longer supported.This was used in
/tests/run.mojo
to run a test suite (which is just one client test for now).The client test worked by running a server in another process (terminal tab) via
mojo lightbug.🔥
, then runningmojo tests/run.mojo
.Now I tried to work around the issue, and moved the test suite to
lightbug.🔥
, which is where the main function for the whole package lives.I'm using the ability to pass a key-value to
mojo run
command like so:mojo -D "TEST" lightbug.🔥
.Currently this doesn't work, getting a bunch of compile errors. I'm assuming this has more to do with the Mojo client code than with the new way to run the main() function, but if anyone has ideas would appreciate.
The text was updated successfully, but these errors were encountered: