fix: add a lesser priority than the default one

[loicmathieu]

Adding a lesser priority than the default one for the NotFoundExceptionMapper allow a user to easily override it.

Be careful that for this to work the exception mapper needs to be of the same parameter type: ExceptionMapper<NotFoundException> if a broader parameter type is used like ExceptionMapper<RuntimeException it will not take precedence from the default one.

Tested on a local project but no test provided.

Refers to #3253

[kenfinnigan]

Is it possible to add an integration test for this?

It would be important to ensure this doesn't regress with other changes

[loicmathieu]

@kenfinnigan I can add a testcase in the resteasy extension with the other test cases but I don't know how to create a test case that runs in dev mode ...

My current test case is as following but it didn't run as dev mode so it didn't implies any regression protection

public class CustomExceptionMapperTestCase {
    @RegisterExtension
    static QuarkusUnitTest runner = new QuarkusUnitTest()
            .setArchiveProducer(() -> ShrinkWrap.create(JavaArchive.class)
                    .addClasses(RootResource.class, CustomExceptionMapper.class));

    @Test
    public void testResourceNotFound() {
        RestAssured.when().get("/not_found")
                .then()
                    .statusCode(200)
                    .body(Matchers.is("not found but OK"));
    }
}

[kenfinnigan]

@stuartwdouglas do we have a way to test DEV mode during the build?

How do we prevent regressions in DEV mode?

[stuartwdouglas]

We have tests in the maven plugin integration tests. They are slow and don't really scale so the plan is to write a dev mode test framework however it is just something that nobody has got to yet.

This is the second time this has come up today so maybe I need to move it up my list.

[loicmathieu]

@stuartwdouglas it's the one from the maven plugin like io.quarkus.maven.it.DevMojoIT ?

It's very complicate to setup (needs a whole project) and I don't think it is really needed to just test this kind of small functionalities.

I propose to skip the test untill we have an easy way to setup test in dev mode via QuarkusUnitTest ...

[stuartwdouglas]

I agree

[stuartwdouglas]

#3317