Fix determining artifact folder for dev profile

[TimJentzsch]

Objective

Closes #218.

For some reason, the profile dev has a target folder named debug instead of dev.
This caused a compile error on native dev builds.

Solution

• Extract the artifact folder path logic into a separate function
• Add a special case for the dev profile
• Add some unit tests

Testing

Run bevy build on a Bevy project, using this branch for the Bevy CLI.
It shouldn't fail with an error.

Future work

We should probably set up some more tests for the CLI, testing all the build and run commands and potentially different targets and profiles.
But that would require non-trivial setup work, so I will defer it for a future task.

[DaAlbrecht]

Nice work! bevy build runs for my test project successfully too.

[DaAlbrecht]

Clippy: We could also unwrap this else block: https://github.com/TheBevyFlock/bevy_cli/blob/main/src/run/mod.rs#L191

[BD103]

Works locally! Merge this when you're ready :)

[TimJentzsch]

Clippy: We could also unwrap this else block: https://github.com/TheBevyFlock/bevy_cli/blob/main/src/run/mod.rs#L191

What do you mean by unwrap here?

[DaAlbrecht]

Clippy: We could also unwrap this else block: https://github.com/TheBevyFlock/bevy_cli/blob/main/src/run/mod.rs#L191

What do you mean by unwrap here?

In the if and elseif branch we early return. Therefore the else statement is redundant. (If we don't return we will hit the else anyway). I think the lsp code action is called "unwrap else block" iirc.

[TimJentzsch]

Clippy: We could also unwrap this else block: https://github.com/TheBevyFlock/bevy_cli/blob/main/src/run/mod.rs#L191

What do you mean by unwrap here?

In the if and elseif branch we early return. Therefore the else statement is redundant. (If we don't return we will hit the else anyway). I think the lsp code action is called "unwrap else block" iirc.

Ah right, because bail! returns. Makes sense!