Skip to content

Conversation

@MartinDelille
Copy link
Member

No description provided.

Copy link
Member Author

@MartinDelille MartinDelille left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok c'est réglé

Copy link
Member Author

@MartinDelille MartinDelille left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour l'instant, j'essaye juste de faire une requête sur l'api publique de github.

pom.xml Outdated

<!-- The repository where to get XWiki dependencies -->
<repositories>
<repository>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il n'y as aucune raison pour que tu ai besoin de ça. Maven central est activé de base dans Maven.

@Role
public interface UserSyncConnector
{
void getUser(String userId);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cette methode n'as pas de sens dans cette API. Si c'est juste pour ton test unitaire il te suffit de caster this.mocker.getComponentUnderTest() en DiscourseUserSyncConnector pour appeller la methode temporaire. Ça évitera d'oublier de la virer à la fin ;)

@MartinDelille
Copy link
Member Author

@tmortagne C'est bon cette première étape est terminée de mon côté. J'ai un problème indépendant mais je vais créer une issue spécifique.

Je te laisse me faire un retour et merger si ça te parait bon!

@MartinDelille
Copy link
Member Author

@tmortagne je viens de pousser ce qu'on a fait en début d'aprèm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants