Comment puis-je maquette JodaTime date réelle?

Je veux tester cette méthode:

 public FirmOrder findActiveByModelColor(ModelColor modelColor) {
   Query query = em.createQuery("FROM FirmOrder fo WHERE fo.modelColor = :modelColor AND fo.year = :year AND fo.month = :month");
   query.setParameter("modelColor", modelColor);
   query.setParameter("year", new DateTime().year().get());
   query.setParameter("month", new DateTime().monthOfYear().get());
   return (FirmOrder) query.getSingleResult();
 }

mais j'ai besoin de DateTime().year().get() et DateTime().dayOfMonth().get() retourne toujours la même date

tks

InformationsquelleAutor Adriano Bacha | 2010-12-15
  1. 52

    Si vous ne pouvez pas ajouter un objet de fabrique comme suggéré par skaffman, vous pouvez utiliser DateTimeUtils.setCurrentMillisFixed().

    • Un commentaire si gentil, je veux comme ça deux fois.
    • Peut-être que vous souhaitez réinitialiser à dos avec système setCurrentMillisSystem après votre test affirmations.
    • Cette API permet de modifier une variable globale, et doit donc être évitée.
    • Avez-vous une autre suggestion?
    InformationsquelleAutor axtavt
  2. 15

    Ensuite, vous devez définir un Clock de l'interface et de l'injecter dans votre classe

    public interface Clock {
    DateTime getCurrentDateTime();
}

    alors:

    Clock clock;

public FirmOrder findActiveByModelColor(ModelColor modelColor) {
   Query query = em.createQuery("FROM FirmOrder fo WHERE fo.modelColor = :modelColor AND fo.year = :year AND fo.month = :month");
   query.setParameter("modelColor", modelColor);
   query.setParameter("year", clock.getCurrentDateTime().year().get());
   query.setParameter("month", clock.getCurrentDateTime().dayOfMonth().get());
   return (FirmOrder) query.getSingleResult();
 }

    Votre test peut alors injecter une mise en œuvre de Clock (par exemple à l'aide d'un moqueur cadre) qui renvoie toujours à un moment fixé.

    - Je utiliser le Clock interface beaucoup dans mes propres trucs, et je reste surpris qu'il ne fait pas partie de l'une des communes de bibliothèques. J'ai deux implémentations j'utilise beaucoup, WallClock et StoppedClock (ce qui est utile pour les tests qui utilisent un temps fixe).

    • J'aime bien ton idée. Mais j'ai des couples de question que je veux poser. Clock#getCurrentDateTime() sera de retour me l'actuelle JodaTime#DateTime, ce qui est excellent. Cependant j'ai besoin de comparer l'état actuel de DateTime à 4PM EST. Dans mon code, j'ai cette fourPM = new DateTime(current.getYear(), current.getMonthOfYear(), current.getDayOfMonth(), 16, 0, 0, 0, DateTimeZone.forID("EST")); pour current = new DateTime()
    • Vous avez besoin de poser une nouvelle question
    • Je viens de créer une nouvelle question. stackoverflow.com/questions/6049777/... s'il vous Plaît aider
    • Que diriez - clock.now()
    • Joda Time fournit l'interface MillisProvider pour cette. L'implémentation par défaut est DateTimeUtils.SYSTEM_MILLIS_PROVIDER
    InformationsquelleAutor skaffman
  3. 4

    Regarde comme la seule option est d'utiliser la fonctionnalité de réponse à la publication de ce commentaire:

    La partie suivante de votre code peut entraîner dur à détecter les erreurs:

    query.setParameter("year", new DateTime().year().get());
query.setParameter("month", new DateTime().monthOfYear().get());

    Imaginons tha aujourd'hui est le dernier jour de l'année 2011 et que cette partie de code est appelé 1 nano seconde avant la nouvelle année et que la première instruction prend plus de 1 nano seconde pour terminer. Cela signifie que l'année sera mis à 2011, mais les mois à 1, mais il avait de meilleur, soit pour l'exercice 2011/12, ou 2012/1.

    Bien que, statistiquement, il est très peu probable que cela arrive, mais logiquement, il peut arriver 🙂

    Vous devez créer une instance de DateTime et l'utiliser pour remplir les deux de year et month.

    • Ok, c'est pas vraiment une réponse à l'OP, mais je upvoted parce que plus les gens doivent être conscients de ce problème. Étant donné que beaucoup d'entreprise magasins (et à la maison set-ups) ne compilation pendant les heures creuses de la nuit (comme 11:30 PM), ce bug est en fait étonnamment commun et ennuyeux. Il est généralement plus subtil, mais c'est pourquoi le StoppedClock mentionné dans une autre réponse est une bonne idée.
    InformationsquelleAutor Behrang
  4. 1

    C'est facile, si l'utilisation de la JMockit Attentes se moquant de l'API:

    @Test
public void findActiveByModelColor()
{
    new NonStrictExpectations()
    {
        @Cascading DateTime dt;

        {
            dt.year().get(); result = 2010;
            dt.monthOfYear().get(); result = 12;
        }
    };

    FirmOrder fo = testedObject.findActiveByModelColor(modelColor);

    //asserts...
}
    • JMockit est excellent au test de code mal écrit.
    • Ainsi, le code qui utilise DateTime (ou java.util.Date) comme c'est toujours mauvais? Ce code à l'aide de l'Apache Commons API Email, qui instancie un SimpleEmail objet et les appels send() sur elle? Est-il mauvais code? Pourquoi?
    InformationsquelleAutor Rogério