Skip to content

Added unit tests for matrix arithmetic #108

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Aug 29, 2019
Merged

Added unit tests for matrix arithmetic #108

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
dd78037
Added unit tests for matrix arithmetic
regexident Aug 6, 2019
3e13863
Renamed existing remaining test functions to match naming scheme
regexident Aug 6, 2019
5597cef
Fix formatting
regexident Aug 6, 2019
489b817
Fixed swiftlint violations
regexident Aug 6, 2019
21caeec
Fixed failing unit test due to large float and resulting decreased ac…
regexident Aug 6, 2019
b2db8c9
Removed unused test function
regexident Aug 11, 2019
eed94e4
Moved computed test values into local `let actual` binding
regexident Aug 11, 2019
964ef19
Added missing unit tests for matrix initializers
regexident Aug 11, 2019
0c8f39e
Fixed unit tests for `Matrix.sum(_:axies:)`
regexident Aug 11, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fixed swiftlint violations
  • Loading branch information
@regexident
regexident committed Aug 29, 2019
commit 489b81757b791db9e572774659c2a5e6519c1892
6 changes: 2 additions & 4 deletions Tests/SurgeTests/MatrixTests.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,8 @@
@testable import Surge
import XCTest

// swiftlint:disable nesting type_body_length

class MatrixTests: XCTestCase {
let floatAccuracy: Float = 1e-8
let doubleAccuracy: Double = 1e-11
Expand Down Expand Up @@ -363,7 +365,6 @@ class MatrixTests: XCTestCase {
}
)

// FIXME: Replace with `**` operator:
XCTAssertEqual(pow(lhs, rhs), expected, accuracy: 1e-5)
}

Expand All @@ -383,7 +384,6 @@ class MatrixTests: XCTestCase {
}
)

// FIXME: Replace with `**` operator:
XCTAssertEqual(pow(lhs, rhs), expected, accuracy: 1e-8)
}

Expand Down Expand Up @@ -429,7 +429,6 @@ class MatrixTests: XCTestCase {
[4, 5, 6],
]

// FIXME: replace with row iterator:
let expected: Matrix<Scalar> = [
[6],
[15],
Expand All @@ -446,7 +445,6 @@ class MatrixTests: XCTestCase {
[4, 5, 6],
]

// FIXME: replace with column iterator:
let expected: Matrix<Scalar> = [
[5, 7, 9],
]
Expand Down