Don't test order of DB results

- reduce flakiness by enabling tests to pass regardless of result order
This commit is contained in:
Rob Mulholand 2018-10-26 14:35:39 -05:00
parent ed624330f6
commit 602802e5f4
2 changed files with 8 additions and 8 deletions

View File

@ -185,8 +185,8 @@ var _ = Describe("Deal Repository", func() {
Expect(err).NotTo(HaveOccurred()) Expect(err).NotTo(HaveOccurred())
Expect(len(missingHeaders)).To(Equal(2)) Expect(len(missingHeaders)).To(Equal(2))
Expect(missingHeaders[0].BlockNumber).To(Equal(startingBlock)) Expect(missingHeaders[0].BlockNumber).To(Or(Equal(startingBlock), Equal(endingBlock)))
Expect(missingHeaders[1].BlockNumber).To(Equal(endingBlock)) Expect(missingHeaders[1].BlockNumber).To(Or(Equal(startingBlock), Equal(endingBlock)))
}) })
It("only treats headers as checked if deal have been checked", func() { It("only treats headers as checked if deal have been checked", func() {
@ -220,8 +220,8 @@ var _ = Describe("Deal Repository", func() {
missingHeadersNode1, err := dealRepository.MissingHeaders(startingBlock, endingBlock) missingHeadersNode1, err := dealRepository.MissingHeaders(startingBlock, endingBlock)
Expect(err).NotTo(HaveOccurred()) Expect(err).NotTo(HaveOccurred())
Expect(len(missingHeadersNode1)).To(Equal(2)) Expect(len(missingHeadersNode1)).To(Equal(2))
Expect(missingHeadersNode1[0].BlockNumber).To(Equal(startingBlock)) Expect(missingHeadersNode1[0].BlockNumber).To(Or(Equal(startingBlock), Equal(endingBlock)))
Expect(missingHeadersNode1[1].BlockNumber).To(Equal(endingBlock)) Expect(missingHeadersNode1[1].BlockNumber).To(Or(Equal(startingBlock), Equal(endingBlock)))
missingHeadersNode2, err := dealRepository2.MissingHeaders(startingBlock, endingBlock) missingHeadersNode2, err := dealRepository2.MissingHeaders(startingBlock, endingBlock)
Expect(err).NotTo(HaveOccurred()) Expect(err).NotTo(HaveOccurred())

View File

@ -189,8 +189,8 @@ var _ = Describe("Dent Repository", func() {
Expect(err).NotTo(HaveOccurred()) Expect(err).NotTo(HaveOccurred())
Expect(len(missingHeaders)).To(Equal(2)) Expect(len(missingHeaders)).To(Equal(2))
Expect(missingHeaders[0].BlockNumber).To(Equal(startingBlock)) Expect(missingHeaders[0].BlockNumber).To(Or(Equal(startingBlock), Equal(endingBlock)))
Expect(missingHeaders[1].BlockNumber).To(Equal(endingBlock)) Expect(missingHeaders[1].BlockNumber).To(Or(Equal(startingBlock), Equal(endingBlock)))
}) })
It("only treats headers as checked if dent has been checked", func() { It("only treats headers as checked if dent has been checked", func() {
@ -223,8 +223,8 @@ var _ = Describe("Dent Repository", func() {
missingHeadersNode1, err := dentRepository.MissingHeaders(startingBlock, endingBlock) missingHeadersNode1, err := dentRepository.MissingHeaders(startingBlock, endingBlock)
Expect(err).NotTo(HaveOccurred()) Expect(err).NotTo(HaveOccurred())
Expect(len(missingHeadersNode1)).To(Equal(2)) Expect(len(missingHeadersNode1)).To(Equal(2))
Expect(missingHeadersNode1[0].BlockNumber).To(Equal(startingBlock)) Expect(missingHeadersNode1[0].BlockNumber).To(Or(Equal(startingBlock), Equal(endingBlock)))
Expect(missingHeadersNode1[1].BlockNumber).To(Equal(endingBlock)) Expect(missingHeadersNode1[1].BlockNumber).To(Or(Equal(startingBlock), Equal(endingBlock)))
missingHeadersNode2, err := dentRepository2.MissingHeaders(startingBlock, endingBlock) missingHeadersNode2, err := dentRepository2.MissingHeaders(startingBlock, endingBlock)
Expect(err).NotTo(HaveOccurred()) Expect(err).NotTo(HaveOccurred())