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

Proper tests #11

Merged
merged 13 commits into from
May 7, 2022
Merged

Proper tests #11

merged 13 commits into from
May 7, 2022

Conversation

rustkas
Copy link
Contributor

@rustkas rustkas commented Apr 23, 2022

  1. Add proper tests for depcache module.
  2. Add launching proper test by Makefile.
  3. Move dializer launching to rebar3 test profile.
  4. Add changes to rebar.config.

Replace 3600 to the constant HOUR
Provide more information about function definitons of depcache API.
Add EDoc options to API documentation (tune CSS styles, add overview file). Add new folder doc_src which contains additional files for documentation generation.
Implement overload option of EDoc.
Show hidden documetation by providing rebar3 option - add additional profile for EDoc private documentation.
Separate some functions to make the code easier to read.
Tunie Makefile: add command for edoc, clean; add new command edoc_private
Refactor some -spec descriptions.
Made some refactoring to explicely transform data.
It is possible that input Key and dict Key is not the some.
Add proper tests for depcache module.
Add launching proper test by Makefile.
Move dializer launching to rebar3 test profile.
Add changes to rebar.config.
@mworrell
Copy link
Member

mworrell commented May 6, 2022

It seems that proper doesn't support OTP 19, I am ok with removing that test target.

@rustkas
Copy link
Contributor Author

rustkas commented May 6, 2022

It seems that proper doesn't support OTP 19, I am ok with removing that test target.

Ok.

The setting of the configuration file that did not work during automatic testing was removed.
Update OTP version list which is supported.
@mworrell
Copy link
Member

mworrell commented May 7, 2022

Maybe just remove the whole OTP 19.3 tests from the .GH tests?

otp_version: [19.3,20.3,21,22,23]

And then, maybe, add OTP 24?

@mworrell
Copy link
Member

mworrell commented May 7, 2022

Just checked manually, and indeed 20.3 and 23 both pass the tests.

@rustkas
Copy link
Contributor Author

rustkas commented May 7, 2022

Maybe just remove the whole OTP 19.3 tests from the .GH tests?

otp_version: [19.3,20.3,21,22,23]

And then, maybe, add OTP 24?

Yes. Of couse.

@mworrell
Copy link
Member

mworrell commented May 7, 2022

Everything passes 🥳 Merging!

And thank you again!

@mworrell mworrell merged commit f2229b3 into zotonic:master May 7, 2022
@mworrell
Copy link
Member

mworrell commented May 7, 2022

Released version 1.9.0

@rustkas rustkas deleted the proper_tests branch May 8, 2022 06:25
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.

None yet

2 participants