Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

API: Wrapper #22

Open
wants to merge 10 commits into
base: working
Choose a base branch
from
Open

API: Wrapper #22

wants to merge 10 commits into from

Conversation

Yaty
Copy link

@Yaty Yaty commented Mar 16, 2018

No description provided.

@MachinisteWeb
Copy link
Member

Je regarde ça semaine prochaine !

Copy link

@forresst forresst left a comment

Choose a reason for hiding this comment

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

Beau travail, quelques remarques et corrections à faire


A `Wrapper` is an object that contains a mounted component or vnode and methods to test the component or vnode.
Un `Wrapper` est un objet qui contient un composant monté, un nœud virtuel et des méthodes pour les tester.

Choose a reason for hiding this comment

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

Un Wrapper est un objet qui contient un composant monté, un nœud virtuel et des méthodes pour les tester.
=>
Un Wrapper est un objet qui contient un composant monté ou un nœud virtuel et des méthodes pour les tester.

`element` `HTMLElement`: the root DOM node of the wrapper
`options` `Object`: Object containing Vue Test Utils options passed to `mount` or `shallow`
`options.attachedToDom` `Boolean`: True if `attachToDom` was passed to `mount` or `shallow`
`vm` `Component`: c'est une instance de Vue. Vous pouvez accéder à toutes les [méthodes et propriétés de l'instance](https://vuejs.org/v2/api/#Instance-Properties) avec `wrapper.vm`. Cela existe uniquement sur les `wrappers` de composants Vue.

Choose a reason for hiding this comment

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

c'est une instance de Vue.
=>
c'est une instance de `Vue`.

`options` `Object`: Object containing Vue Test Utils options passed to `mount` or `shallow`
`options.attachedToDom` `Boolean`: True if `attachToDom` was passed to `mount` or `shallow`
`vm` `Component`: c'est une instance de Vue. Vous pouvez accéder à toutes les [méthodes et propriétés de l'instance](https://vuejs.org/v2/api/#Instance-Properties) avec `wrapper.vm`. Cela existe uniquement sur les `wrappers` de composants Vue.
`element` `HTMLElement`: le nœud principal du DOM du `wrapper`.

Choose a reason for hiding this comment

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

le nœud principal racine du DOM du

@@ -1,10 +1,10 @@
# `emittedByOrder()`

<p><strong>⚠Cette page est actuellement en cours de traduction française. Vous pouvez repasser plus tard ou <a href="https://github.com/vuejs-fr/vue-test-utils" target="_blank">participer à la traduction</a> de celle-ci dès maintenant !</strong></p><p>Return an Array containing custom events emitted by the `Wrapper` `vm`.</p>
Retourne un tableau contenant des évènements émis par l'instance de Vue `vm` de `Wrapper`.

Choose a reason for hiding this comment

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

contenant des évènements émis
=>
contenant des évènements émis personnalisés


Returns false if called on an empty `Wrapper` or `WrapperArray`.
Retourne `false` si appellé sur un `Wrapper` ou `WrapperArray` vide.

Choose a reason for hiding this comment

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

appellé => appelé


<p><strong>⚠Cette page est actuellement en cours de traduction française. Vous pouvez repasser plus tard ou <a href="https://github.com/vuejs-fr/vue-test-utils" target="_blank">participer à la traduction</a> de celle-ci dès maintenant !</strong></p><p>Sets `Wrapper` `vm` props and forces update.</p>
Fixe les `props` de l'instance de Vue `vm` du `Wrapper` `vm` et force la mise à jour.

Choose a reason for hiding this comment

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

Fixe => Assigne


<p><strong>⚠Cette page est actuellement en cours de traduction française. Vous pouvez repasser plus tard ou <a href="https://github.com/vuejs-fr/vue-test-utils" target="_blank">participer à la traduction</a> de celle-ci dès maintenant !</strong></p><p>Sets `Wrapper` `vm` props and forces update.</p>
Fixe les `props` de l'instance de Vue `vm` du `Wrapper` `vm` et force la mise à jour.
Sets `Wrapper` `vm` props and forces update.

Choose a reason for hiding this comment

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

Cette ligne doit être supprimée

<p><strong>⚠Cette page est actuellement en cours de traduction française. Vous pouvez repasser plus tard ou <a href="https://github.com/vuejs-fr/vue-test-utils" target="_blank">participer à la traduction</a> de celle-ci dès maintenant !</strong></p><p>Sets `Wrapper` `vm` props and forces update.</p>
Fixe les `props` de l'instance de Vue `vm` du `Wrapper` `vm` et force la mise à jour.
Sets `Wrapper` `vm` props and forces update.
**Note : le `Wrapper` doit contenir une instance de Vue.**

**Note the Wrapper must contain a Vue instance.**

Choose a reason for hiding this comment

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

Cette ligne et quelques sauts de lignes doivent être supprimées

@@ -1,18 +1,18 @@
# `update()`

<p><strong>⚠Cette page est actuellement en cours de traduction française. Vous pouvez repasser plus tard ou <a href="https://github.com/vuejs-fr/vue-test-utils" target="_blank">participer à la traduction</a> de celle-ci dès maintenant !</strong></p><p>Force root Vue component to re-render.</p>
Force le composant Vue principal à se mettre à jour.

Choose a reason for hiding this comment

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

Force le composant Vue principal à se mettre à jour.
=>
Force le composant racine de Vue à se mettre à jour.


If called on a `Wrapper` containing a `vm`, it will force the `Wrapper` `vm` to re-render.
Si appellée sur un `Wrapper` contenant une instance de Vue `vm`, il la mettra à jour.

Choose a reason for hiding this comment

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

Si ~~appellée~~~~ appelée sur un Wrapper contenant une instance de Vue vm, il la mettra à jour cela forcera le Wrapper à la re-rendre.

@Yaty
Copy link
Author

Yaty commented Mar 19, 2018

Merci, je corrige ça assez rapidement :)

@Yaty
Copy link
Author

Yaty commented Mar 27, 2018

Good ! Merci !

@MachinisteWeb
Copy link
Member

Merci @Yaty. Un peu débordé actuellement, je garde mon temps pour aligner mes dépôts Vue, je reviens participer au review ici aussi tôt que mon planning se décante un peu !

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

Successfully merging this pull request may close these issues.

3 participants