NeTEx v0.2.1

Utilise le profil `pan:french_profile:1` qui introduit des règles spécifiques au
profil France.

Ce profil reprend les règles du profil `enroute:starter-kit` par copie. Ce
profil est configuré à froid par un script.

Author: ptitfred

apps/transport/lib/validators/enroute_chouette_valid_client.ex

@@ -4,7 +4,8 @@ defmodule Transport.EnRouteChouetteValidClient.Wrapper do
   Documentation: https://documenter.getpostman.com/view/9950294/2sA3e2gVEE
   """
 
-  @callback create_a_validation(Path.t()) :: binary()
+  @callback create_a_validation(filepath :: Path.t()) :: binary()
+  @callback create_a_validation(filepath :: Path.t(), rule_set :: binary()) :: binary()
   @callback get_a_validation(binary()) ::
               :pending
               | {:successful, binary(), integer()}
@@ -25,11 +26,11 @@ defmodule Transport.EnRouteChouetteValidClient do
   @base_url "https://chouette-valid.enroute.mobi/api/validations"
 
   @impl Transport.EnRouteChouetteValidClient.Wrapper
-  def create_a_validation(filepath) do
+  def create_a_validation(filepath, rule_set \\ "enroute:starter-kit") do
     form =
       {:multipart,
        [
-         {"validation[rule_set]", "enroute:starter-kit"},
+         {"validation[rule_set]", rule_set},
          {"validation[include_schema]", "true"},
          make_file_part("validation[file]", filepath)
        ]}

apps/transport/lib/validators/netex/CHANGELOG.md

@@ -16,3 +16,10 @@ without any specific inline documentation.
 Categories are used to build the summary of a NeTEx validation. Each category
 has a title, a description (providing links to any relevant documentation), and
 some hints for usual errors (optional).
+
+## 0.2.1
+
+- Uses the profile `pan:french_profile:1` (which includes rules from
+    `enroute:starter-kit`)
+  - new rules introduced to check specific rules of the French profile
+      (mandatory attributes)

apps/transport/lib/validators/netex/validator.ex

@@ -6,7 +6,7 @@ defmodule Transport.Validators.NeTEx.Validator do
 
   require Logger
   alias Transport.Jobs.NeTExPollerJob, as: Poller
-  alias Transport.Validators.NeTEx.ResultsAdapters.V0_2_0, as: ResultsAdapter
+  alias Transport.Validators.NeTEx.ResultsAdapters.V0_2_1, as: ResultsAdapter
 
   @behaviour Transport.Validators.Validator
 
@@ -32,7 +32,7 @@ defmodule Transport.Validators.NeTEx.Validator do
   def validator_name, do: "enroute-chouette-netex-validator"
 
   # This will change with an actual versioning of the validator
-  def validator_version, do: "0.2.0"
+  def validator_version, do: "0.2.1"
 
   @impl Transport.Validators.Validator
   def validate_and_save(%DB.ResourceHistory{} = resource_history) do
@@ -205,7 +205,7 @@ defmodule Transport.Validators.NeTEx.Validator do
     setup_validation(filepath) |> poll_validation_results(0)
   end
 
-  defp setup_validation(filepath), do: client().create_a_validation(filepath)
+  defp setup_validation(filepath), do: client().create_a_validation(filepath, "pan:french_profile:1")
 
   def poll_validation_results(validation_id, retries) do
     case client().get_a_validation(validation_id) do

apps/transport/test/support/enroute_chouette_valid_client_helpers.ex

@@ -4,10 +4,10 @@ defmodule Transport.Test.EnRouteChouetteValidClientHelpers do
   """
   import Mox
 
-  def expect_create_validation do
+  def expect_create_validation(ruleset) do
     validation_id = with_running_validation()
 
-    expect(Transport.EnRouteChouetteValidClient.Mock, :create_a_validation, fn _ -> validation_id end)
+    expect(Transport.EnRouteChouetteValidClient.Mock, :create_a_validation, fn _, ^ruleset -> validation_id end)
 
     validation_id
   end

apps/transport/test/transport/jobs/netex_poller_job_test.exs

@@ -56,7 +56,7 @@ defmodule Transport.Jobs.NeTExPollerJobTest do
 
     assert multi_validation.command == "http://localhost:9999/chouette-valid/#{validation_id}"
     assert multi_validation.validator == "enroute-chouette-netex-validator"
-    assert multi_validation.validator_version == "0.2.0"
+    assert multi_validation.validator_version == "0.2.1"
     assert multi_validation.result == %{}
     assert multi_validation.metadata.metadata == %{"retries" => attempts, "elapsed_seconds" => duration}
   end
@@ -77,7 +77,7 @@ defmodule Transport.Jobs.NeTExPollerJobTest do
 
     assert multi_validation.command == "http://localhost:9999/chouette-valid/#{validation_id}/messages"
     assert multi_validation.validator == "enroute-chouette-netex-validator"
-    assert multi_validation.validator_version == "0.2.0"
+    assert multi_validation.validator_version == "0.2.1"
     assert multi_validation.metadata.metadata == %{"retries" => attempts, "elapsed_seconds" => duration}
 
     assert multi_validation.result == %{

apps/transport/test/transport/jobs/on_demand_netex_poller_job_test.exs

@@ -53,7 +53,7 @@ defmodule Transport.Test.Transport.Jobs.OnDemandNeTExPollerJobTest do
                       result: nil,
                       validation_timestamp: date,
                       validator: "enroute-chouette-netex-validator",
-                      validator_version: "0.2.0"
+                      validator_version: "0.2.1"
                     } = validation |> DB.Repo.reload() |> DB.Repo.preload(:metadata)
 
              assert DateTime.diff(date, DateTime.utc_now()) <= 1
@@ -75,7 +75,7 @@ defmodule Transport.Test.Transport.Jobs.OnDemandNeTExPollerJobTest do
              result: %{},
              validation_timestamp: date,
              validator: "enroute-chouette-netex-validator",
-             validator_version: "0.2.0"
+             validator_version: "0.2.1"
            } = validation |> DB.Repo.reload() |> DB.Repo.preload(:metadata)
 
     assert DateTime.diff(date, DateTime.utc_now()) <= 1

apps/transport/test/transport/jobs/on_demand_validation_job_test.exs

@@ -492,14 +492,14 @@ defmodule Transport.Test.Transport.Jobs.OnDemandValidationJobTest do
   end
 
   def expect_netex_with_errors(messages) do
-    validation_id = expect_create_validation()
+    validation_id = expect_create_validation("pan:french_profile:1")
     expect_failed_validation(validation_id, 10)
 
     expect_get_messages(validation_id, messages)
   end
 
   def expect_netex_long_lasting do
-    expect_create_validation() |> expect_pending_validation()
+    expect_create_validation("pan:french_profile:1") |> expect_pending_validation()
   end
 
   defp run_job(%DB.MultiValidation{} = validation) do

apps/transport/test/transport/validators/netex/validator_test.exs

@@ -45,23 +45,23 @@ defmodule Transport.Validators.NeTEx.ValidatorTest do
     test "valid NeTEx" do
       resource_history = mk_netex_resource()
 
-      validation_id = expect_create_validation() |> expect_successful_validation(12)
+      validation_id = expect_create_validation("pan:french_profile:1") |> expect_successful_validation(12)
 
       assert :ok == Validator.validate_and_save(resource_history)
 
       multi_validation = load_multi_validation(resource_history.id)
 
       assert multi_validation.command == "http://localhost:9999/chouette-valid/#{validation_id}"
       assert multi_validation.validator == "enroute-chouette-netex-validator"
-      assert multi_validation.validator_version == "0.2.0"
+      assert multi_validation.validator_version == "0.2.1"
       assert multi_validation.result == %{}
       assert multi_validation.metadata.metadata == %{"retries" => 0, "elapsed_seconds" => 12}
     end
 
     test "pending validation" do
       resource_history = mk_netex_resource()
 
-      validation_id = expect_create_validation() |> expect_pending_validation()
+      validation_id = expect_create_validation("pan:french_profile:1") |> expect_pending_validation()
 
       assert :ok == Validator.validate_and_save(resource_history)
 
@@ -79,7 +79,7 @@ defmodule Transport.Validators.NeTEx.ValidatorTest do
     test "invalid NeTEx" do
       resource_history = mk_netex_resource()
 
-      validation_id = expect_create_validation() |> expect_failed_validation(31)
+      validation_id = expect_create_validation("pan:french_profile:1") |> expect_failed_validation(31)
 
       expect_get_messages(validation_id, @sample_error_messages)
 
@@ -89,7 +89,7 @@ defmodule Transport.Validators.NeTEx.ValidatorTest do
 
       assert multi_validation.command == "http://localhost:9999/chouette-valid/#{validation_id}/messages"
       assert multi_validation.validator == "enroute-chouette-netex-validator"
-      assert multi_validation.validator_version == "0.2.0"
+      assert multi_validation.validator_version == "0.2.1"
       assert multi_validation.metadata.metadata == %{"retries" => 0, "elapsed_seconds" => 31}
 
       assert multi_validation.result == %{
@@ -136,7 +136,7 @@ defmodule Transport.Validators.NeTEx.ValidatorTest do
     test "valid NeTEx" do
       resource_url = mk_raw_netex_resource()
 
-      expect_create_validation() |> expect_successful_validation(9)
+      expect_create_validation("pan:french_profile:1") |> expect_successful_validation(9)
 
       assert {:ok, %{"validations" => %{}, "metadata" => %{retries: 0, elapsed_seconds: 9}}} ==
                Validator.validate(resource_url)
@@ -145,7 +145,7 @@ defmodule Transport.Validators.NeTEx.ValidatorTest do
     test "invalid NeTEx" do
       resource_url = mk_raw_netex_resource()
 
-      validation_id = expect_create_validation() |> expect_failed_validation(25)
+      validation_id = expect_create_validation("pan:french_profile:1") |> expect_failed_validation(25)
 
       expect_get_messages(validation_id, @sample_error_messages)
 
@@ -188,7 +188,7 @@ defmodule Transport.Validators.NeTEx.ValidatorTest do
     test "pending" do
       resource_url = mk_raw_netex_resource()
 
-      validation_id = expect_create_validation() |> expect_pending_validation()
+      validation_id = expect_create_validation("pan:french_profile:1") |> expect_pending_validation()
 
       assert {:pending, validation_id} == Validator.validate(resource_url)
     end