PipingSoilProfilesImporterTest.cs

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Introduced StochasticSoilModelCollection which is an ObservableList of StochasticSoilModel with an added property for the last known file path. Set last known file path from the importer when import is successful. Renamed PipingSoilProfilesImporter to StochasticSoilModelImporter to better match its responsibility. Used string interpolation and nameof. Inlined resources in tests.

Related to Issue [WTI-1080]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@7682 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: d516d97af58d226b9e94f0d78a9b4548f05ed432

Former-commit-id: 2f9f840fa4c706712e251ccbf92d97fa7af8461b

    • -722
    • +0
    ./PipingSoilProfilesImporterTest.cs
  1. … 16 more files in changeset.
Incorporated comments review 1385: (Editorial) - Made the spelling of "Canceled" or "Uncancelable (opposed to "Cancelled" or "Uncancellable") consistent throughout the application.

Note:

"Cancellation" is legit U.S. spelling and therefore unchanged. "Cancelation" is not common and rarely used.

Related to Issue [WTI-490]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@7669 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 665e259f03237740e63d6441088354043dae52f2

Former-commit-id: 98c98ae04c081843c7bab2c55946744a8d06dad2

  1. … 35 more files in changeset.
Incorporated comments review 1385: - Improved FileImporterBase: -- Use null propagation -- Removed unnecessary setter -- Virtual method to abstract method, fixed affected failing tests - Improved StructuresImporter: -- Used nameof function -- Renamed private field parameter name conform coding standards - Minor editorial changes - Removed redundant aliases

Related to Issue [WTI-490]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@7668 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: ae36ef10b17db21a36e1c42d2fff77760a8665d6

Former-commit-id: 125e339e8568945268976ffc402eba124269dfff

  1. … 8 more files in changeset.
- Added NotifyProgress calls when adding data to the data model - Synchronised message values across all the importers

Note:

No tests are available to verify the cancel behaviour for the FeatureBasedMapData because it has no NotifyProgress (and consequently no SetProgress handle to set the Action it should execute during the import)

Related to Issue [WTI-490]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@7655 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: ee82e9d839a1acfffb0ccb4ff219a30856c744e7

Former-commit-id: a79f8f2e64e63d1e9bac5672052dc8bc143bf91f

  1. … 18 more files in changeset.
Implemented Reuse tests in the concrete implementations of the FileImporterBase Related to Issue [WTI-490]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@7650 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: d766c4944786409b114582323cf947d60e99337f

Former-commit-id: 0ff0cb1b31a88afc319a093050888e49cd2428f5

  1. … 5 more files in changeset.
Merged revision(s) 7595-7638 from branches/WTI-490:

Merged revision(s) 7537-7594 from trunk

........

Initial commit of changes from kleijn:

- Removed HandleUserCancellingImport: FileImporter should not decide on its own how to handle Canceled actions. Consequently removed the messages that are generated by these handles

- Added logic to the importers: Cancelling is not possible when putting data on the data model. Action will always execute when it is in this stage

- Added branching paths in the activity: Import() returns false when it has failed or is cancelled

Note: failing tests will be fixed in following commits

........

Fixed tests of the FileImporterBase and Activity:

- Checks on the states are made for the 4 different paths of the Import:

-- Import successful, canceled or not

-- Import not successful, canceled or not

- Moved logging from the activity to the importer

- Added tests to assert mentioned behaviour

........

Fixed tests of the PipingSurfaceLinesCsvImporter:

- Added test to verify behaviour when canceling at the two stages during the import

- Reverted changes related to HandleUserCancellingImport: messages are written by the LogImportCanceledMessage and handled adequately by the base

- Added resource value for the ImportActivity (missing file commit)

........

Fixed tests of the PipingSoilProfilesImporter:

- Added test to verify behaviour when canceling at the three stages during the import

- Reverted changes related to HandleUserCancellingImport: messages are written by the LogImportCanceledMessage and handled adequately by the base

- Cleaned redundant brackets

........

Fixed tests of the StructuresImporter:

- Added test to verify behaviour when canceling at the two stages during the import. Test regarding cancelling while Creating structures is missing

- Reverted changes related to HandleUserCancellingImport: messages are written by the LogImportCanceledMessage and handled adequately by the base

Adjusted test in PipingSoilProfilesImporterTest

- Changed initial value of the result to make clear that the value is set correctly

........

Fixed tests of the ProfilesImporters:

- Added test to verify behaviour when canceling at the two stages during the import. Test regarding cancelling while Creating structures is missing

- Reverted changes related to HandleUserCancellingImport: messages are written by the LogImportCanceledMessage and handled adequately by the base

- Removed obsolete Cancel call in the PipingSoilProfilesImporterTest

........

FileImporters should call Observers in case the import was successfull AND was cancelled at an uncancellable stage.

Note: test is missing to verify this behaviour

........

Fixed tests of the ProfilesImporters:

- Reverted changes related to HandleUserCancellingImport: messages are written by the LogImportCanceledMessage and handled adequately by the base

- Temporarily removed the Cancel tests, because the FeatureBasedMapDataImporter does not have a handle for NotifyProgress. Hence, Cancel method cannot be invoked when Import is running.

........

Fixed tests of the FailureMechanismSectionsImporter:

- Added test to verify behaviour when canceling at the three stages during the import

- Reverted changes related to HandleUserCancellingImport: messages are written by the LogImportCanceledMessage and handled adequately by the base

- Cleaned up tests in the StructuresImporterTest

........

Fixed tests of the ReferenceLineImporter:

- Added test to verify behaviour when canceling at the two stages during the import

- Reverted changes related to HandleUserCancellingImport: messages are written by the LogImportCanceledMessage and handled adequately by the base

........

Related to Issue [WTI-490]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@7646 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 1c355702dfbd410c7c4733529d8db9e243496eba

Former-commit-id: 75c614862d018947936ef85e3775a125c425c032

    • -11
    • +105
    ./PipingSoilProfilesImporterTest.cs
  1. … 26 more files in changeset.
Fixed tests of the ProfilesImporters: - Added test to verify behaviour when canceling at the two stages during the import. Test regarding cancelling while Creating structures is missing - Reverted changes related to HandleUserCancellingImport: messages are written by the LogImportCanceledMessage and handled adequately by the base

- Removed obsolete Cancel call in the PipingSoilProfilesImporterTest

Related to Issue [WTI-490]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/branches/WTI-490@7632 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 686bdd815ed5479367b69fb431a59ef8fc0df1a0

Former-commit-id: cf9ed26e6a2def68a2919f8324bb8ad59154109b

  1. … 6 more files in changeset.
Fixed tests of the StructuresImporter: - Added test to verify behaviour when canceling at the two stages during the import. Test regarding cancelling while Creating structures is missing - Reverted changes related to HandleUserCancellingImport: messages are written by the LogImportCanceledMessage and handled adequately by the base

Adjusted test in PipingSoilProfilesImporterTest

- Changed initial value of the result to make clear that the value is set correctly

Related to Issue [WTI-490]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/branches/WTI-490@7630 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: e2edd56f05d391faac7470b1bb75dd6f40ae025f

Former-commit-id: e741664a290587da194a47ee9a587e542c76df02

  1. … 4 more files in changeset.
Fixed tests of the PipingSoilProfilesImporter: - Added test to verify behaviour when canceling at the three stages during the import - Reverted changes related to HandleUserCancellingImport: messages are written by the LogImportCanceledMessage and handled adequately by the base - Cleaned redundant brackets Related to Issue [WTI-490]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/branches/WTI-490@7620 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: abd77e13efd2eb809d6cb6c117aaa59c65c65d46

Former-commit-id: f0404e75a098165b1f9b664e5d8a02b9d6ee08ba

    • -8
    • +103
    ./PipingSoilProfilesImporterTest.cs
  1. … 2 more files in changeset.
Initial commit of changes from kleijn: - Removed HandleUserCancellingImport: FileImporter should not decide on its own how to handle Canceled actions. Consequently removed the messages that are generated by these handles - Added logic to the importers: Cancelling is not possible when putting data on the data model. Action will always execute when it is in this stage - Added branching paths in the activity: Import() returns false when it has failed or is cancelled

Note: failing tests will be fixed in following commits

Related to Issue [WTI-490]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/branches/WTI-490@7598 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 409bfee1e3deebb4fe4368cb7010a8c4c37dae5e

Former-commit-id: 98175da77f6910f259097a90626bbb10450e2c8f

  1. … 20 more files in changeset.
used lower case after colon when using FileReaderErrorMessageBuilder. Added FirstToLower method in StringExtensions. Related to Issue [WTI-948]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@6693 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: e9d3cccc3d91f565fc53bdfffabb2449c517437e

Former-commit-id: b80c97fbcc39c0000d0348fb5fa4591371f53bed

  1. … 58 more files in changeset.
* Added standard serialization constructor to all Exception classes. * Explicitly defined culture for string.Format methods. * Replaced IFileImporter.ProgressChanged property to method (recommendation by C# coding conventions) * Renamed classes containing win32.dll invokes to NativeMethods (recommendation by C# coding conventions)

Related to Issue [WTI-931]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@6471 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 78fae4fb038935c7a58a0fb534baa750b173d589

Former-commit-id: fc7d0938a3677be66218915a3e665a9cc6757df9

    • -52
    • +26
    ./PipingSoilProfilesImporterTest.cs
  1. … 164 more files in changeset.
Improved some resources Related to Issue [WTI-832]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/branches/Release/16_1_1@5063 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 3b265054a01504ae5770026c97d33da126cc1361

Former-commit-id: c9a560a9a3d201b6cc01d7b6b1682885956acf13

  1. … 5 more files in changeset.
Improved some resources Related to Issue [WTI-832]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/branches/Release/16_1_1@5063 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: e78acf062dab11fea25f689d43fcf96b046e0400

Former-commit-id: 0896db64ee71455e02faabc85753623cf8136fb5

  1. … 5 more files in changeset.
Improved some resources Related to Issue [WTI-832]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/branches/Release/16_1_1@5055 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 4faddc4f2bd6c86f55f6a87f5b5400349622eb51

Former-commit-id: 20e0e8c11461cbf7b53016477becbd9ec216506c

  1. … 5 more files in changeset.
Improved some resources Related to Issue [WTI-832]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/branches/Release/16_1_1@5055 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 44b3c5f0edaee59fed37fdad0bea8fe193e80674

Former-commit-id: 1ead8fc920842a7dc4e12ac2387b48c86d0dc447

  1. … 5 more files in changeset.
* FileImporterBase now takes the import targets as constructor argument. ** Removed targetItem argument from IFileImporter.DoPostImportUpdates. * Removed abstract keyword from FileImportBase.ProgressChanged. Related to Issue [WTI-497]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@5033 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 5a81cae977dee513511a3257dedbeba6df57983a

Former-commit-id: cc770d424d622685b68b2d59f2c6fe21d1cd949c

  1. … 17 more files in changeset.
* Moved filePath argument from IFileImporter.Import to import class constructor. Related to Issue [WTI-497]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@5023 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 7c21b2cb0e14ce82b05211488ea6dd8b8f3ef8a4

Former-commit-id: 38d3653a05f0b36eb7d42bd93a7b8a1f4b44f453

    • -40
    • +36
    ./PipingSoilProfilesImporterTest.cs
  1. … 29 more files in changeset.
* Removed unsued properties and methods from IFileImporter. ** Added 'name' argument to FileImportActivity constructor. * Removed 'targetItem' from IFileImport.Import method. * Refactored GuiImportHandler to work with importInfo. Related to Issue [WTI-497]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@5016 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: bdc6bd3e00fb0ed24703b91de78e974e322b8cdf

Former-commit-id: 52be75d229dca9b85b83ee556983e03267adadf1

    • -71
    • +19
    ./PipingSoilProfilesImporterTest.cs
  1. … 24 more files in changeset.
* Migrated PipingSoilProfileImporter to ImportInfo

Related to Issue [WTI-497]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@5005 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 86393d1f0de55ef8dcfeeb5d044edd0c5d0cb834

Former-commit-id: e39f1f200a71301ac8a5ae43a3a05d3fa99d3f14

    • -233
    • +62
    ./PipingSoilProfilesImporterTest.cs
  1. … 6 more files in changeset.
Merged revision(s) 4850, 4857, 4861 from branches/Release/16_1_1: - Remove strange assembly in resource. - Updated UnusedResourceSearcher.exe Related to Issue WTI 744

........

* Improved resource name to match coding conventions.

Related to Issue WTI 798

........

Use GetFullPath to get a proper path for the SaveFileDialog

........

Merged revision(s) 4837, 4842, 4846 from branches/Release/16_1_1:

Checked VerifyAll for every ReplayAll call.

Whenever encountered: mocking of implementation classes replaced with instances.

Reviewed by: kleijn

........

Added test for RingtoetsPipingSurfaceLineProperties.

........

Corrected documentation.

Related to Issue WTI 737

........

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@4865 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: df400900af18215aa45d4c4535e15722f2daea3f

Former-commit-id: 8dccb6e105b15f43d5b135c05c7a3d4b50febed7

    • -12
    • +3
    ./PipingSoilProfilesImporterTest.cs
  1. … 58 more files in changeset.
Checked VerifyAll for every ReplayAll call. Whenever encountered: mocking of implementation classes replaced with instances. Reviewed by: kleijn

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/branches/Release/16_1_1@4837 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 29a0064808fc56f7668060fa079a23a4af0013e5

Former-commit-id: c751a2248b677164cb164f72e59d5171cedf17d2

    • -12
    • +3
    ./PipingSoilProfilesImporterTest.cs
  1. … 51 more files in changeset.
Checked VerifyAll for every ReplayAll call. Whenever encountered: mocking of implementation classes replaced with instances. Reviewed by: kleijn

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/branches/Release/16_1_1@4837 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 66ebd56c5332e0f0871ddcc2659a1fb3806e4d11

Former-commit-id: 4f66d3e88f7d71c30ddc212c8de0818408899373

    • -12
    • +3
    ./PipingSoilProfilesImporterTest.cs
  1. … 51 more files in changeset.
* Added licence headers to all projects (except the installer, as this is not compatible with the tool we're using) Related to Issue [WTI-737]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@4724 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: b979849273917beaa200a788771088b6fa3df5b0

Former-commit-id: b23b99557c064e7da02709783a30facae89b69db

    • -1
    • +22
    ./PipingSoilProfilesImporterTest.cs
  1. … 310 more files in changeset.
* Improved readability of tests * Turned String into string Related to Issue [WTI-757]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@4612 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 701ea98de739f0f328ead01558e5e443c904e706

Former-commit-id: cdb77cc67acfc2869c510cc8a1f32c0201a5f40e

  1. … 141 more files in changeset.
Removed unused AbovePhreaticLevel and DryUnitWeight properties from readers and storing classes. Removed unused EffectiveStress property from the IEffectiveThicknessCalculator and implementors. Due to stochastic parameters, effective stress needs to be calculated in Ringtoets. Related to Issue [WTI-686]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@4135 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 9cac85375a4bdfec0c70b252aaf48d2f3b31789c

Former-commit-id: 60f9628656712e9fdd83984776cb0bfd987c8dee

    • -20
    • +0
    ./PipingSoilProfilesImporterTest.cs
  1. … 29 more files in changeset.
Added parameters for distributions of stochastic parameters to PipingSoilLayer. Added validation for optional parameters for distributions. Related to Issue [WTI-686]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@4111 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 878d4c35f36b5842fb500c80c0a004b165490927

Former-commit-id: a06270109dff8ce8bb93b338a7273c8ffa922a97

  1. … 36 more files in changeset.
Fixed bug where incorrect number of 2D profiles was returned. Updated complete.soil so that also 2D profiles will be imported from it. Updated tests accordingly. Related to Issue [WTI-137]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@3865 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 153942d06c7176bc735f03308a20c35df38f1a1e

Former-commit-id: 066336ac3518259f60cbb3d3cbd8db61a818b35d

  1. … 4 more files in changeset.
Turned StochasticSoilModelContext into ObservableWrappedObjectContextBase Related to Issue [WTI-543]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@3511 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: c812e4ebed6aa604cad310c70db03bfb1c79ae7e

Former-commit-id: 3b91a3c295e75be9ddc6db814664358cebee6e32

    • -32
    • +31
    ./PipingSoilProfilesImporterTest.cs
  1. … 8 more files in changeset.
Rename "faalmechanisme" to "toetsspoor" Related to Issue [WTI-508]

git-svn-id: https://repos.deltares.nl/repos/WettelijkToetsInstrumentarium/trunk@3157 5a3db67b-de53-47b5-99c8-a1c30a6650e2

Former-commit-id: 1c5693dbda7b4eb83e864f4ddd0713e963a40a79

Former-commit-id: 9e7331fa24b741f34cb3c730085a816fa9d7bf3b

  1. … 21 more files in changeset.