Merge #122

bors[bot] · P1n3appl3 · bors[bot] · commit e979c7d82ba7 · 2018-07-17T03:19:09.000Z
122: Add sanity_check for atan2 r=japaric a=P1n3appl3 It's already been merged, but now I can say with certainty that this closes #9 Co-authored-by: Joseph Ryan <josephryan3.14@gmail.com>
diff --git a/src/math/atan2.rs b/src/math/atan2.rs
@@ -70,3 +70,13 @@ pub fn atan2(y: f64, x: f64) -> f64 {
         _ => (z - PI_LO) - PI, /* atan(-,-) */
     }
 }
+
+#[test]
+fn sanity_check() {
+    assert_eq!(atan2(0.0, 1.0), 0.0);
+    assert_eq!(atan2(0.0, -1.0), PI);
+    assert_eq!(atan2(-0.0, -1.0), -PI);
+    assert_eq!(atan2(3.0, 2.0), atan(3.0 / 2.0));
+    assert_eq!(atan2(2.0, -1.0), atan(2.0 / -1.0) + PI);
+    assert_eq!(atan2(-2.0, -1.0), atan(-2.0 / -1.0) - PI);
+}

Original file line number	Diff line number	Diff line change
`@@ -70,3 +70,13 @@ pub fn atan2(y: f64, x: f64) -> f64 {`
`70`	`70`	`_ => (z - PI_LO) - PI, /* atan(-,-) */`
`71`	`71`	`}`
`72`	`72`	`}`
	`73`	`+`
	`74`	`+#[test]`
	`75`	`+fn sanity_check() {`
	`76`	`+ assert_eq!(atan2(0.0, 1.0), 0.0);`
	`77`	`+ assert_eq!(atan2(0.0, -1.0), PI);`
	`78`	`+ assert_eq!(atan2(-0.0, -1.0), -PI);`
	`79`	`+ assert_eq!(atan2(3.0, 2.0), atan(3.0 / 2.0));`
	`80`	`+ assert_eq!(atan2(2.0, -1.0), atan(2.0 / -1.0) + PI);`
	`81`	`+ assert_eq!(atan2(-2.0, -1.0), atan(-2.0 / -1.0) - PI);`
	`82`	`+}`