Skip to content

Commit

Permalink
updated test workflow with code coverage summary
Browse files Browse the repository at this point in the history
  • Loading branch information
StephenHodgson committed Nov 19, 2023
1 parent e53e04a commit f3e4754
Show file tree
Hide file tree
Showing 2 changed files with 26 additions and 6 deletions.
19 changes: 17 additions & 2 deletions .github/workflows/Publish-Nuget.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,24 +31,39 @@ env:
jobs:
build:
if: ${{ !github.event_name == 'pull_request' || !github.event.pull_request.draft }}
runs-on: windows-latest
runs-on: linux-latest

steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0

- uses: actions/setup-dotnet@v3
with:
dotnet-version: ${{ env.DOTNET_VERSION }}

- uses: microsoft/setup-msbuild@v1

- name: Test Packages
run: dotnet test --configuration Release
if: ${{ github.ref != 'refs/heads/main' && github.event_name != 'push' }}
run: dotnet test --configuration Release --verbosity normal --collect:"XPlat Code Coverage" --results-directory ./coverage
env:
OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
OPENAI_ORGANIZATION_ID: ${{ secrets.OPENAI_ORGANIZATION_ID }}

- name: Code Coverage Summary Report
if: ${{ github.ref != 'refs/heads/main' && github.event_name != 'push' }}
uses: irongut/[email protected]
with:
filename: 'coverage/*/coverage.cobertura.xml'
badge: true
format: 'markdown'
output: 'both'

- name: Write Coverage Job Summary
if: ${{ github.ref != 'refs/heads/main' && github.event_name != 'push' }}
run: cat code-coverage-results.md >> $GITHUB_STEP_SUMMARY

- name: Build Pack and Publish NuGet Package
run: |
$projectPath = "${{ github.workspace }}\OpenAI-DotNet"
Expand Down
13 changes: 9 additions & 4 deletions OpenAI-DotNet-Tests/TestFixture_12_Threads.cs
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ public async Task Test_04_01_CreateMessage()
{
Assert.IsFalse(string.IsNullOrWhiteSpace(testThreadId));
Assert.IsNotNull(OpenAIClient.ThreadsEndpoint);
var file = await CreateFileForAssistant();
var file = await CreateFileForAssistantAsync();

var request = new CreateMessageRequest("Test content",
new[] { file.Id },
Expand Down Expand Up @@ -143,8 +143,8 @@ public async Task Test_04_03_ModifyMessage()
public async Task Test_04_04_ListMessageFiles()
{
Assert.IsFalse(string.IsNullOrWhiteSpace(testThreadId));
var file1 = await CreateFileForAssistant();
var file2 = await CreateFileForAssistant();
var file1 = await CreateFileForAssistantAsync();
var file2 = await CreateFileForAssistantAsync();
var createRequest = new CreateMessageRequest("Test content", new[] { file1.Id, file2.Id });
var message = await OpenAIClient.ThreadsEndpoint.CreateMessageAsync(testThreadId, createRequest);
var list = await OpenAIClient.ThreadsEndpoint.ListFilesAsync(message.ThreadId, message.Id);
Expand Down Expand Up @@ -172,7 +172,7 @@ public async Task Test_05_DeleteThread()
Console.WriteLine($"Deleted thread {testThreadId}");
}

private async Task<FileData> CreateFileForAssistant()
private async Task<FileData> CreateFileForAssistantAsync()
{
var testData = "Some useful knowledge";
var fileName = "test.txt";
Expand All @@ -181,5 +181,10 @@ private async Task<FileData> CreateFileForAssistant()
var file = await OpenAIClient.FilesEndpoint.UploadFileAsync(fileName, "assistants");
return file;
}

private async Task CleanupFilesAsync()
{
Task.CompletedTask;
}
}
}

0 comments on commit f3e4754

Please sign in to comment.