openlmis-file-upload

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Revert "Revert "Revert "OLMIS-6261: Commented out openlmisFileUpload directive tests"""

This reverts commit e0891f181f385b44f05948004c904dfb9dd93435.

    • -2
    • +2
    ./openlmis-file-upload.directive.spec.js
Revert "Revert "OLMIS-6261: Commented out openlmisFileUpload directive tests""

This reverts commit f756089a1f3166c2119b9e4926b6b4a41355adc3.

    • -2
    • +2
    ./openlmis-file-upload.directive.spec.js
Revert "Revert "OLMIS-6261: Refactored unit tests (part 1)""

This reverts commit ce9f844fb5944aeb644055d35d0b201f4eae8110.

    • -41
    • +35
    ./openlmis-file-upload.directive.spec.js
  1. … 46 more files in changeset.
Revert "OLMIS-6261: Refactored unit tests (part 1)"

This reverts commit a92390f79fb4dd1c1d8ca9e9823320ff03e014c7.

    • -35
    • +41
    ./openlmis-file-upload.directive.spec.js
  1. … 46 more files in changeset.
Revert "OLMIS-6261: Commented out openlmisFileUpload directive tests"

This reverts commit 27bfe8833aa37cc406a8bddfe74af75d5971c3ad.

    • -2
    • +2
    ./openlmis-file-upload.directive.spec.js
OLMIS-6261: Commented out openlmisFileUpload directive tests

    • -2
    • +2
    ./openlmis-file-upload.directive.spec.js
Could we separate this html fragment of code into a variable just like it is in other tests?

Could we separate this html fragment of code into a variable just like it is in other tests?

Could we separate this html fragment of code into a variable just like it is in other tests?

Could we separate this html fragment of code into a variable just like it is in other tests?

is this change expected?

is this change expected?

is this change expected?

is this change expected?

is this change expected?

is this change expected?

OLMIS-6261: Refactored unit tests (part 1)
OLMIS-6261: Refactored unit tests (part 1)
OLMIS-6261: Refactored unit tests (part 1)

    • -41
    • +35
    ./openlmis-file-upload.directive.spec.js
  1. … 46 more files in changeset.
Or create a ticket. Currently, the test is not executed so it similar to situation where test does not exist

Or create a ticket. Currently, the test is not executed so it similar to situation where test does not exist

Should I just remove it?

Should I just remove it?

Do we really need a commented out test? Do we have a ticket to fix this test?

Do we really need a commented out test? Do we have a ticket to fix this test?

Most of these are auto fixes.

Most of these are auto fixes.

OLMIS-4388: Fixed ESlint tech debt for specs
OLMIS-4388: Fixed ESlint tech debt for specs
OLMIS-4388: Fixed ESlint tech debt for specs

    • -8
    • +8
    ./openlmis-file-upload.directive.spec.js
  1. … 88 more files in changeset.
Fixed in FEOLMIS-3291.

Fixed in FEOLMIS-3291.

The PerfectScrollbar is not defined anywhere, it's a global. We always want to wrap such dependencies into an angular constant.

The PerfectScrollbar is not defined anywhere, it's a global. We always want to wrap such dependencies into an angular constant.

this should be in dev-ui

this should be in dev-ui

why this is failing?

why this is failing?

Fixed some tech debt and standardized formating in preparation for updated .eslintrc file
Fixed some tech debt and standardized formating in preparation for updated .eslintrc file