Skip to content

Commit

Permalink
Fixed minor build warnings. (Removed unused references/dead code.)
Browse files Browse the repository at this point in the history
  • Loading branch information
Peter-Schorn committed May 26, 2024
1 parent d3e87f0 commit 9cc938b
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 10 deletions.
3 changes: 0 additions & 3 deletions Sources/SpotifyWebAPI/API/SpotifyAPI+Utilities.swift
Original file line number Diff line number Diff line change
Expand Up @@ -205,9 +205,6 @@ public extension SpotifyAPI {
// Avoid Error:
// Swift runtime failure: Stride size must not be zero

// the offset of the page after the current one
let lowerBoundOffset = page.offset + page.limit

let pageOffsets = generatePageOffsets(
page,
maxExtraPages: maxExtraPages
Expand Down
14 changes: 7 additions & 7 deletions Tests/SpotifyAPIMainTests/API Tests/SpoifyAPIArtistTests.swift
Original file line number Diff line number Diff line change
Expand Up @@ -79,13 +79,13 @@ extension SpotifyAPIArtistTests {
XCTAssertNil(albums.next)
XCTAssertNil(albums.previous)

let expectedAlbumsHREFString = """
https://api.spotify.com/v1/artists/4kSGbjWGxTchKpIxXPJv0B\
/albums?offset=0&limit=35&include_groups=album,single,\
compilation,appears_on&market=US
"""
let expectedAlbumsHREF = URL(string: expectedAlbumsHREFString)!
.sortedQueryItems()
// let expectedAlbumsHREFString = """
// https://api.spotify.com/v1/artists/4kSGbjWGxTchKpIxXPJv0B\
// /albums?offset=0&limit=35&include_groups=album,single,\
// compilation,appears_on&market=US
// """
// let expectedAlbumsHREF = URL(string: expectedAlbumsHREFString)!
// .sortedQueryItems()

// XCTAssert(
// albums.href.sortedQueryItems().absoluteString.starts(
Expand Down

0 comments on commit 9cc938b

Please sign in to comment.