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

Unit tests #75

Open
playXE opened this issue Jul 6, 2021 · 8 comments
Open

Unit tests #75

playXE opened this issue Jul 6, 2021 · 8 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@playXE
Copy link
Collaborator

playXE commented Jul 6, 2021

We do have a few unit tests already but they do not actually cover full engine

@playXE playXE added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Jul 6, 2021
@andrieshiemstra
Copy link
Contributor

Do you want unit tests in a separate mod or in comments?

Personally i tend to make them in comments (give every public fn a # example section) so they get deleted/changed with the function

@playXE
Copy link
Collaborator Author

playXE commented Jul 7, 2021

TBH I don't know. Some of the code might need much more tests that could fit in the doc comment i.e object methods that might have lots and lost of different behaviours

@andrieshiemstra
Copy link
Contributor

FYI, working on a CONTRIBUTING.md to explain how we want tests and such

Question: if we add docs to public API and such: sagudev once made me a github action which publishes the rustdoc at github. maybe that would be nice for starlight also?
see:
https://github.com/HiRoFa/quickjs_es_runtime/blob/master/.github/workflows/rust.yml#L82

docs here

That way you can have the latest docs available online without publishing to crates.io

@playXE
Copy link
Collaborator Author

playXE commented Jul 7, 2021

Github action docs look nice! There's also docs/ directory in the root of the repository, shall we consider moving docs from there to Rust generated docs?

@andrieshiemstra
Copy link
Contributor

shall we consider moving docs from there to Rust generated docs?

agreed!

@jameslahm
Copy link
Member

I think we could also make a public website using some static site generator, for example Docusaurus. And the docs under the root directory can be there. Also it could include a reference link to the cargo generated docs. At last, the website and the reference docs can all be deployed on GitHub pages using actions.

@playXE
Copy link
Collaborator Author

playXE commented Jul 7, 2021

@jameslahm there's already one: https://starlight-js.github.io/ but I haven't updated it for a while. I want to believe that soon I'll update it and write a few blog posts about internals,GC etc

@jameslahm
Copy link
Member

@playXE I see it before. I mean we could also consider some other more functional static site generator, including support search, table of content, auto translation to multiple language etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants