Integration test for MySQL is not configured correctly #2460

Closed
opened 2025-11-02 04:37:06 -06:00 by GiteaMirror · 2 comments
Owner

Originally created by @typeless on GitHub (Oct 28, 2018).

Not sure if it is deliberate or not, the test-mysql tag in .drone.yml is declared twice. As a result, it seems that the integration test for MySQL does not run. What has been run is actually make integration-test-coverage.

Originally created by @typeless on GitHub (Oct 28, 2018). Not sure if it is deliberate or not, the `test-mysql` tag in `.drone.yml` is declared twice. As a result, it seems that the integration test for MySQL does not run. What has been run is actually `make integration-test-coverage`.
GiteaMirror added the type/testing label 2025-11-02 04:37:06 -06:00
Author
Owner

@techknowlogick commented on GitHub (Oct 29, 2018):

One is only run when the branch is master, and the other is run several other cases (see when part of the yaml for more details).

Now, I'm not sure if that is deliberate (why only mysql in that case), and so I will leave this open in case anyone has insight into it.

Thank you for raising this ticket.

@techknowlogick commented on GitHub (Oct 29, 2018): One is only run when the branch is master, and the other is run several other cases (see `when` part of the yaml for more details). Now, I'm not sure if that is deliberate (why only mysql in that case), and so I will leave this open in case anyone has insight into it. Thank you for raising this ticket.
Author
Owner

@lafriks commented on GitHub (Nov 13, 2018):

Yes it is intended so as when pushed to master and for pulls it calculate test coverage but for tags not

@lafriks commented on GitHub (Nov 13, 2018): Yes it is intended so as when pushed to master and for pulls it calculate test coverage but for tags not
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#2460