Skip to content

Commit

Permalink
#24 Fixed lyrics search for musixmatch
Browse files Browse the repository at this point in the history
  • Loading branch information
skuill committed Aug 19, 2024
1 parent 93e91cb commit c18c4b5
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

<ItemGroup>
<PackageReference Include="Moq" Version="4.18.2" />
<PackageReference Include="System.Text.Json" Version="7.0.2" />
<PackageReference Include="System.Text.Json" Version="8.0.4" />
</ItemGroup>

<ItemGroup>
Expand Down
6 changes: 3 additions & 3 deletions LyricsScraperNET/LyricsScraperNET.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -28,16 +28,16 @@

<ItemGroup>
<PackageReference Include="Genius.NET" Version="4.0.1" />
<PackageReference Include="HtmlAgilityPack" Version="1.11.58" />
<PackageReference Include="HtmlAgilityPack" Version="1.11.63" />
<PackageReference Include="Microsoft.Extensions.Caching.Abstractions" Version="6.0.0" />
<PackageReference Include="Microsoft.Extensions.Caching.Memory" Version="6.0.1" />
<PackageReference Include="Microsoft.Extensions.Configuration.Abstractions" Version="6.0.0" />
<PackageReference Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="6.0.0" />
<PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="6.0.2" />
<PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="6.0.0" />
<PackageReference Include="MusixmatchClientLib" Version="1.1.5" />
<PackageReference Include="MusixmatchClientLib" Version="1.1.8" />
<PackageReference Include="Sigil" Version="5.0.0" />
<PackageReference Include="System.Text.Json" Version="7.0.2" />
<PackageReference Include="System.Text.Json" Version="8.0.4" />
</ItemGroup>

<ItemGroup>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,10 +13,10 @@
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
</PackageReference>
<PackageReference Include="System.Text.Json" Version="7.0.2" />
<PackageReference Include="xunit" Version="2.5.0" />
<PackageReference Include="System.Text.Json" Version="8.0.4" />
<PackageReference Include="xunit" Version="2.9.0" />
<PackageReference Include="xunit.MSTest" Version="1.0.3" />
<PackageReference Include="xunit.runner.visualstudio" Version="2.5.0">
<PackageReference Include="xunit.runner.visualstudio" Version="2.8.2">
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
</PackageReference>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ namespace LyricsScraperNET.IntegrationTest.Providers.Musixmatch
{
public class MusixmatchProviderTest : ProviderTestBase
{
[Theory (Skip="Bug with the infinite requests loop. Should be enabled after https://github.com/skuill/LyricsScraperNET/issues/24")]
[Theory]
[MemberData(nameof(GetTestData), parameters: "Providers\\Musixmatch\\lyric_test_data.json")]
public void SearchLyric_IntegrationDynamicData_Success(LyricsTestData testData)
{
Expand All @@ -31,7 +31,7 @@ public void SearchLyric_IntegrationDynamicData_Success(LyricsTestData testData)
Assert.False(searchResult.Instrumental);
}

[Theory(Skip = "Bug with the infinite requests loop. Should be enabled after https://github.com/skuill/LyricsScraperNET/issues/24")]
[Theory]
[MemberData(nameof(GetTestData), parameters: "Providers\\Musixmatch\\instrumental_test_data.json")]
public void SearchLyric_IntegrationDynamicData_Instrumental(LyricsTestData testData)
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,10 @@
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
</PackageReference>
<PackageReference Include="System.Text.Json" Version="7.0.2" />
<PackageReference Include="xunit" Version="2.4.1" />
<PackageReference Include="System.Text.Json" Version="8.0.4" />
<PackageReference Include="xunit" Version="2.9.0" />
<PackageReference Include="xunit.MSTest" Version="1.0.3" />
<PackageReference Include="xunit.runner.visualstudio" Version="2.4.3">
<PackageReference Include="xunit.runner.visualstudio" Version="2.8.2">
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
</PackageReference>
Expand Down

0 comments on commit c18c4b5

Please sign in to comment.