openlmis-contract-tests

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
OLMIS-5493, add redis and ensure it's available for referencedata

removed misleading method

removed misleading method

added

added

removed

removed

Revert "OLMIS-5415: Waited one second before retrieving location"

This reverts commit 63196557549877be38e0b01149ebf9fbd46ff905.

The name might be a little misleading. Wouldn't it make more sense to name it something telling us we're doing a request here?

The name might be a little misleading. Wouldn't it make more sense to name it something telling us we're doing a request here?

I think that this hides an important problem (201 being returned before the object is created). This information should definitely be propagated up and investigated.

I think that this hides an important problem (201 being returned before the object is created). This information should definitely be propagated up and investigated.

no, because we don't set id in the URL (the first parameter) and we use POST which basically is used to create objects. I created this private method to avoid code duplication.

no, because we don't set id in the URL (the first parameter) and we use POST which basically is used to create objects. I created this private method to avoid code duplication.

To make sure that a location on the FHIR server will be created. On CI there were situations where one request returns 201 status code but next one - which retrieve the object - returns 404 status ...

To make sure that a location on the FHIR server will be created. On CI there were situations where one request returns 201 status code but next one - which retrieve the object - returns 404 status code. I didn't have this locally so I thought that maybe we do something too fast in tests

This could use a Javadoc as the other newly-added exception have.

This could use a Javadoc as the other newly-added exception have.

Does this mean we will be able to update with this method aswell?

Does this mean we will be able to update with this method aswell?

Nevermind, I can see now.

Nevermind, I can see now.

Do we need this?

Do we need this?

What is it for?

What is it for?

OLMIS-5493, add Redis for ReferenceData

done

done

OLMIS-5415: Waited one second before retrieving location

Could you remove comma after 'and'?

Could you remove comma after 'and'?

OLMIS-5415: Added validations to avoid 4XX and 5XX errors
OLMIS-5415: Added validations to avoid 4XX and 5XX errors
3.4.1 release
3.4.1 release
Version freeze

Update fulfillment to v 8.0.0-SNAPSHOT