fix: change redirect route arguments

[GSpaki]

Description

When setting arguments using RedirectRoute, a ModularArguments is passed instead of the actual argument.

This caused the following error:
The following _TypeError was thrown building Builder:
type 'ModularArguments' is not a subtype of type 'XPTO'

Checklist

• The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
• I have read the Contributor Guide and followed the process outlined for submitting PRs.
• I have updated/added tests for ALL new/updated/fixed functionality.
• I have updated/added relevant documentation in docs and added dartdoc comments with ///.
• I have updated/added relevant examples in examples.

Breaking Change

• Yes, this is a breaking change.
• No, this is not a breaking change.

Related Issues

[toshiossada]

não entendi a modificaçao, so mudou o nome da variavel?

[GSpaki]

não entendi a modificaçao, so mudou o nome da variavel?

@toshiossada
Não, mudei de arguments pra modularArguments pra tentar deixar um pouco mais claro que não é o argumento propriamente dito, esse fica no modularArguments.data.

O select route espera o argumento da rota, não o modularArgument
Por exemplo, se usasse uma string de argumento junto com o redirectRoute, não era passada para rota a string em si, mas um modularArguments com um .data com a string

O que daria erro ao tratar o argumento => type 'ModularArguments' is not a subtype of type 'String'

Por isso no selectRoute agora passa modularArguments.data em vez de apenas modularArguments

[toshiossada]

pode criar um test para esse cenário?

[GSpaki]

pode criar um test para esse cenário?

Eu tentei kkkkk
Eu não compreendo bem o funcionamento do modular pra criar um teste que verifique o parâmetro usando o RedirectRoute. No caso de teste que eu modifiquei, estava mockado o retorno do getArgument.
Precisaria usar o redirectRoute e pegar o argumento real passado, sem mock

Foi mal pela falta da descrição melhor e do teste, eu nunca tinha tentado contribuir pra um package.
Vou tentar ver dnv se consigo o caso de teste

[GSpaki]

@jacobaraujo7 isso ainda entra no modular 5? Ou entrou no 6?
Talvez eu tenha apontado pro lugar errado o merge.
Tô com um projeto que precisaria disso

[allanwolski]

This is not working with dynamic routes.