Don't return 404 if .cargo-ok file exists

This was a regression introduced in #692.

Author: jyn514

src/web/source.rs

@@ -115,7 +115,7 @@ impl FileList {
 
                     // skip .cargo-ok generated by cargo
                     if path == ".cargo-ok" {
-                        return None;
+                        continue;
                     }
 
                     // look only files for req_path
@@ -261,6 +261,7 @@ pub fn source_browser_handler(req: &mut Request) -> IronResult<Response> {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use crate::test::{assert_success, wrapper};
     use serde_json::json;
 
     #[test]
@@ -306,4 +307,20 @@ mod tests {
 
         assert_eq!(correct_json, serde_json::to_value(&file_list).unwrap(),);
     }
+
+    #[test]
+    fn cargo_ok_not_skipped() {
+        wrapper(|env| {
+            let db = env.db();
+            db.fake_release()
+                .name("fake")
+                .version("0.1.0")
+                .source_file(".cargo-ok", b"ok")
+                .source_file("README.md", b"hello")
+                .create()?;
+            let web = env.frontend();
+            assert_success("/crate/fake/0.1.0/source/", web)?;
+            Ok(())
+        });
+    }
 }