forgejo

Commit Graph

Author	SHA1	Message	Date
Aravinth Manivannan	ade41f3f04	tests: add coverage for models migration helpers (#18162 ) They were previously not covered at all, either by integration tests or unit tests. This PR also fixes a bug where the `num_comments` field was incorrectly set to include all types of comments. It sets num_closed_issues: 0 as default in milestone unit test fixtures. If they are not set, Incr("num_closed_issues") will be a noop because the field is null.	3 years ago

Author

SHA1

Message

Date

Aravinth Manivannan

ade41f3f04

tests: add coverage for models migration helpers (#18162 )

They were previously not covered at all, either by integration tests or unit tests.

This PR also fixes a bug where the `num_comments` field was incorrectly set to include all types of comments.

It sets num_closed_issues: 0 as default in milestone unit test fixtures. If they are not set, Incr("num_closed_issues") will be a noop because the field is null.

1 Commits (72cced1f3e3500b01705c9e9cdbdc59e9a27cb1c)