fix: param nesting for report creation requests (closes #658) (#659)

# Description

The params were not nested at the root level for report creation which
is required so params like `columns` and `additional_columns` were not
accepted at the API. This PR corrects the nesting to put them at the
root which should fix these params when generating reports. (closes
#658)

Additionally, un-nested all cassette files. Years ago we tested this lib
against multiple frameworks at once which required multiple sets of
cassettes which is no longer the case. The cassettes nested in a `net`
folder is no longer helpful, moving them up a level.

Finally, cleaned up a similar problem with webhooks that lingered from
years earlier where to avoid collisions, we had to differentiate the
urls used in webhook generation based on net version in use. Removed
that as a part of this too.

<!-- Please provide a general summary of your PR changes and link any
related issues or other pull requests. -->

# Testing

Re-recorded all affected cassettes (otherwise cassettes just moved
location).

<!--
Please provide details on how you tested this code. See below.

- All pull requests must be tested (unit tests where possible with
accompanying cassettes, or provide a screenshot of end-to-end testing
when unit tests are not possible)
- New features must get a new unit test
- Bug fixes/refactors must re-record existing cassettes 
-->

# Pull Request Type

Please select the option(s) that are relevant to this PR.

- [x] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] Improvement (fixing a typo, updating readme, renaming a variable
name, etc)

Author: Justintime50

CHANGELOG.md

@@ -4,7 +4,7 @@
 using EasyVCR;
 
 // ReSharper disable once CheckNamespace
-namespace EasyPost.Integration.Utilities
+namespace EasyPost.Integration
 {
     public class Utils
     {
@@ -74,21 +74,6 @@ internal static string ReadFile(string path)
             return File.ReadAllText(filePath);
         }
 
-        internal static string NetVersion
-        {
-            get
-            {
-                // ReSharper disable once RedundantAssignment
-                // ReSharper disable once ConvertToConstant.Local
-                string netVersion = "net";
-#if NET472
-                netVersion = "netstandard";
-#endif
-
-                return netVersion;
-            }
-        }
-
         // ReSharper disable once InconsistentNaming
         public class VCR
         {
@@ -123,9 +108,6 @@ public VCR(string? testCassettesFolder = null, ApiKey apiKey = ApiKey.Test)
 
                 _testCassettesFolder = Path.Combine(GetSourceFileDirectory(), CassettesFolder); // create "cassettes" folder in same directory as test files
 
-                string netVersionFolder = NetVersion;
-
-                _testCassettesFolder = Path.Combine(_testCassettesFolder, netVersionFolder); // create .NET version-specific folder in "cassettes" folder
 
                 if (testCassettesFolder != null)
                 {

EasyPost.Integration/TestUtils.cs

@@ -101,7 +101,7 @@ public async Task TestRetrieveAllPayloads()
             UseVCR("retrieve_payloads_for_event");
 
             // Create a webhook to receive the event
-            string url = $"https://example.com/all_payloads/{TestUtils.NetVersion}";
+            string url = "https://example.com/all_payloads";
             Webhook webhook = await Client.Webhook.Create(new Dictionary<string, object> { { "url", url } });
             CleanUpAfterTest(webhook.Id);
 
@@ -129,7 +129,7 @@ public async Task TestRetrievePayload()
             UseVCR("retrieve_payload_for_event");
 
             // Create a webhook to receive the event
-            string url = $"https://example.com/single_payload/{TestUtils.NetVersion}";
+            string url = $"https://example.com/single_payload";
             Webhook webhook = await Client.Webhook.Create(new Dictionary<string, object> { { "url", url } });
             CleanUpAfterTest(webhook.Id);
 

…ion/cassettes/net/synchronous/async.json …gration/cassettes/synchronous/async.jsonEasyPost.Integration/cassettes/net/synchronous/async.json renamed to EasyPost.Integration/cassettes/synchronous/async.json EasyPost.Integration/cassettes/net/synchronous/async.json renamed to EasyPost.Integration/cassettes/synchronous/async.json

@@ -40,7 +40,7 @@ public async Task TestCreate()
         {
             UseVCR("create");
 
-            string url = $"https://example.com/create/{TestUtils.NetVersion}";
+            string url = $"https://example.com/create";
 
             Dictionary<string, object> webhookParams = new Dictionary<string, object>()
             {
@@ -79,7 +79,7 @@ public async Task TestRetrieve()
         {
             UseVCR("retrieve");
 
-            string url = $"https://example.com/retrieve/{TestUtils.NetVersion}";
+            string url = $"https://example.com/retrieve";
 
             Webhook webhook = await Client.Webhook.Create(new Dictionary<string, object> { { "url", url } });
             CleanUpAfterTest(webhook.Id);
@@ -159,7 +159,7 @@ public async Task TestUpdate()
         {
             UseVCR("update");
 
-            string url = $"https://example.com/update/{TestUtils.NetVersion}";
+            string url = $"https://example.com/update";
 
             Webhook webhook = await Client.Webhook.Create(new Dictionary<string, object> { { "url", url } });
             CleanUpAfterTest(webhook.Id);
@@ -188,7 +188,7 @@ public async Task TestDelete()
         {
             UseVCR("delete");
 
-            string url = $"https://example.com/delete/{TestUtils.NetVersion}";
+            string url = $"https://example.com/delete";
 
             Webhook webhook = await Client.Webhook.Create(new Dictionary<string, object> { { "url", url } });
             CleanUpAfterTest(webhook.Id);

…tes/net/synchronous/via_get_awaiter.json …ssettes/synchronous/via_get_awaiter.jsonEasyPost.Integration/cassettes/net/synchronous/via_get_awaiter.json renamed to EasyPost.Integration/cassettes/synchronous/via_get_awaiter.json EasyPost.Integration/cassettes/net/synchronous/via_get_awaiter.json renamed to EasyPost.Integration/cassettes/synchronous/via_get_awaiter.json

@@ -40,7 +40,7 @@ public async Task TestCreate()
         {
             UseVCR("create");
 
-            string url = $"https://example.com/beta/create/{TestUtils.NetVersion}";
+            string url = $"https://example.com/beta/create";
 
             Dictionary<string, object> data = new Dictionary<string, object> { { "url", url } };
 
@@ -80,7 +80,7 @@ public async Task TestUpdate()
         {
             UseVCR("update");
 
-            string url = $"https://example.com/beta/update/{TestUtils.NetVersion}";
+            string url = $"https://example.com/beta/update";
             Parameters.Webhook.Create webhookParameters = new()
             {
                 Url = url,