Review requirement IDs used in tests

[beeme1mr]

Requirements

The requirement IDs used in the specification changed when we added support for client-side feature flags. The .NET SDK uses those IDs in some tests. These should be reviewed to make sure they match the spec.

Requirements that are out of date:

• dotnet-sdk/test/OpenFeature.Tests/OpenFeatureClientTests.cs
  • Spec IDs: 1.4.5-1.4.14
• dotnet-sdk/test/OpenFeature.Tests/OpenFeatureHookTests.cs
  • Spec IDs: 4.3.3-4.3.8

Here's the diff between spec v0.5.2 and v0.8.0.

[roelofb]

The evaluation context also lacks the required 'targeting key` field

dotnet-sdk/test/OpenFeature.Tests/OpenFeatureEvaluationContextTests.cs

Line 46 in 2c237df

[Specification("3.1.1", "The `evaluation context` structure MUST define an optional `targeting key` field of type string, identifying the subject of the flag evaluation.")]

[toddbaert]

@roelofb excellent catch! I've created a bug for this: #230

[roelofb]

@toddbaert Added a PR: #231