Skip to content

Commit

Permalink
Merge branch 'master' into dev
Browse files Browse the repository at this point in the history
  • Loading branch information
fuzzzerd committed Sep 11, 2018
2 parents 8769f66 + 4270a14 commit 6d1e734
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion src/FMData.Xml/FileMakerXmlClient.cs
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ public async override Task<ICreateResponse> SendAsync<T>(ICreateRequest<T> req)

// append fileName to request since thats not represented in the request itself
var httpRequestContent = new StringContent(requestContent + $"&-db={_fileName}");

// execute the request by posting to fms
var response = await _client.PostAsync(url, httpRequestContent);

Expand Down
2 changes: 0 additions & 2 deletions tests/FMData.Rest.Tests/FindRequestTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -451,7 +451,6 @@ public async Task ProcessContainerData_Should_Skip_InvalidUris(string uri)
Assert.Null(model.SomeContainerFieldData);
}


[Fact]
public async Task FindAsync_WithPortals_ShouldHaveData()
{
Expand Down Expand Up @@ -484,7 +483,6 @@ public async Task FindAsync_WithPortals_ShouldHaveData()
Assert.Equal(16, response.First().Actions.First().ID);
}


[Fact]
public async Task SendAsync_Dictionary_WithPortals_ShouldHaveData()
{
Expand Down

4 comments on commit 6d1e734

@fuzzzerd
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fuzzzerd
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to look into this for #32

@fuzzzerd
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fuzzzerd
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to handle this in #32

Please sign in to comment.