Add a doctor command to fix inconsistent run status

[Zettat123]

#35783 fixes an actions rerun bug. Due to this bug, some runs may be incorrectly marked as StatusWaiting even though all the jobs are in done status. These runs cannot be run or cancelled. This PR adds a new doctor command to fix the inconsistent run status.

gitea doctor check --run fix-actions-unfinished-run-status --fix

Thanks to @ChristopherHX for the test.

[ChristopherHX]

I would want to let cancellation succeed if a user requested it in this case, proposal here: #35842 . Backport is bigger step, so this doctor command is good.

With my test data of #35842, this PR works

func TestActionsCancelStuckJobDoctor(t *testing.T) {
	defer tests.PrepareTestEnv(t)()

	for _, check := range doctor.Checks {
		if check.Name != "fix-actions-unfinished-run-status" {
			continue
		}
		check.Run(t.Context(), log.GetLogger(log.DEFAULT), true)

		// check if the run is cancelled by id
		run := unittest.AssertExistsAndLoadBean(t, &actions_model.ActionRun{
			ID: 805,
		})
		assert.Equal(t, actions_model.StatusCancelled, run.Status)
	}
}

I think having tests for this doctor command is generally better.

[Zettat123]

I think having tests for this doctor command is generally better.

Thank you for the test data. I added test in 9f100d1

[wxiaoguang]

Usually end users have no idea about the "doctor" commands.

Is it possible to make the "runs" self-healing? For example: cancel it via UI?

I believe adding more doctor commands is not a proper approach.

[Zettat123]

Usually end users have no idea about the "doctor" commands.

Is it possible to make the "runs" self-healing? For example: cancel it via UI?

I believe adding more doctor commands is not a proper approach.

Yes. This PR is used to help users quickly fix the incorrect run status. #35842 will allow users to cancel the run even if all jobs are done