From 222cd46a57a074f0ac179c7382ca370159dbcd59 Mon Sep 17 00:00:00 2001
From: Tom Hughes <tom@compton.nu>
Date: Wed, 7 Mar 2012 21:12:02 +0000
Subject: [PATCH] Add some more unit tests for traces

---
 test/unit/trace_test.rb    | 83 +++++++++++++++++++++++++++++++++++++-
 test/unit/tracetag_test.rb | 23 ++++++++++-
 2 files changed, 104 insertions(+), 2 deletions(-)

diff --git a/test/unit/trace_test.rb b/test/unit/trace_test.rb
index db6dc6044..b8cf6b88a 100644
--- a/test/unit/trace_test.rb
+++ b/test/unit/trace_test.rb
@@ -6,5 +6,86 @@ class TraceTest < ActiveSupport::TestCase
   def test_trace_count
     assert_equal 5, Trace.count
   end
-  
+
+  def test_visible
+    check_query(Trace.visible, :public_trace_file, :anon_trace_file, :trackable_trace_file, :identifiable_trace_file)
+  end
+
+  def test_visible_to
+    check_query(Trace.visible_to(1), :public_trace_file, :identifiable_trace_file)
+    check_query(Trace.visible_to(2), :public_trace_file, :anon_trace_file, :trackable_trace_file, :identifiable_trace_file)
+    check_query(Trace.visible_to(3), :public_trace_file, :identifiable_trace_file)
+  end
+
+  def test_public
+    check_query(Trace.public, :public_trace_file, :identifiable_trace_file, :deleted_trace_file)
+  end
+
+  def test_validations
+    trace_valid({})
+    trace_valid({:user_id => nil}, false)
+    trace_valid({:name => 'a'*255})
+    trace_valid({:name => 'a'*256}, false)
+    trace_valid({:description => nil}, false)
+    trace_valid({:description => 'a'*255})
+    trace_valid({:description => 'a'*256}, false)
+    trace_valid({:visibility => "private"})
+    trace_valid({:visibility => "public"})
+    trace_valid({:visibility => "trackable"})
+    trace_valid({:visibility => "identifiable"})
+    trace_valid({:visibility => "foo"}, false)
+  end
+
+  def test_tagstring
+    trace = Trace.new
+    trace.tagstring = "foo bar baz"
+    assert_equal 3, trace.tags.length
+    assert_equal "foo", trace.tags[0].tag
+    assert_equal "bar", trace.tags[1].tag
+    assert_equal "baz", trace.tags[2].tag
+    assert_equal "foo, bar, baz", trace.tagstring
+    trace.tagstring = "foo, bar baz ,qux"
+    assert_equal 3, trace.tags.length
+    assert_equal "foo", trace.tags[0].tag
+    assert_equal "bar baz", trace.tags[1].tag
+    assert_equal "qux", trace.tags[2].tag
+    assert_equal "foo, bar baz, qux", trace.tagstring
+  end
+
+  def test_public?
+    assert_equal true, gpx_files(:public_trace_file).public?
+    assert_equal false, gpx_files(:anon_trace_file).public?
+    assert_equal false, gpx_files(:trackable_trace_file).public?
+    assert_equal true, gpx_files(:identifiable_trace_file).public?
+    assert_equal true, gpx_files(:deleted_trace_file).public?
+  end
+
+  def test_trackable?
+    assert_equal false, gpx_files(:public_trace_file).trackable?
+    assert_equal false, gpx_files(:anon_trace_file).trackable?
+    assert_equal true, gpx_files(:trackable_trace_file).trackable?
+    assert_equal true, gpx_files(:identifiable_trace_file).trackable?
+    assert_equal false, gpx_files(:deleted_trace_file).trackable?
+  end
+
+  def test_identifiable?
+    assert_equal false, gpx_files(:public_trace_file).identifiable?
+    assert_equal false, gpx_files(:anon_trace_file).identifiable?
+    assert_equal false, gpx_files(:trackable_trace_file).identifiable?
+    assert_equal true, gpx_files(:identifiable_trace_file).identifiable?
+    assert_equal false, gpx_files(:deleted_trace_file).identifiable?
+  end
+
+private
+
+  def check_query(query, *traces)
+    traces = traces.map { |t| gpx_files(t) }.sort
+    assert_equal traces, query.order(:id)
+  end
+
+  def trace_valid(attrs, result = true)
+    entry = Trace.new(gpx_files(:public_trace_file).attributes, :without_protection => true)
+    entry.assign_attributes(attrs, :without_protection => true)
+    assert_equal result, entry.valid?, "Expected #{attrs.inspect} to be #{result}"
+  end
 end
diff --git a/test/unit/tracetag_test.rb b/test/unit/tracetag_test.rb
index 41e77dfe4..2112e5891 100644
--- a/test/unit/tracetag_test.rb
+++ b/test/unit/tracetag_test.rb
@@ -6,5 +6,26 @@ class TracetagTest < ActiveSupport::TestCase
   def test_tracetag_count
     assert_equal 1, Tracetag.count
   end
-  
+
+  def test_validations
+    tracetag_valid({})
+    tracetag_valid({:tag => nil}, false)
+    tracetag_valid({:tag => ''}, false)
+    tracetag_valid({:tag => 'a'})
+    tracetag_valid({:tag => 'a'*255})
+    tracetag_valid({:tag => 'a'*256}, false)
+    tracetag_valid({:tag => 'a/b'}, false)
+    tracetag_valid({:tag => 'a;b'}, false)
+    tracetag_valid({:tag => 'a.b'}, false)
+    tracetag_valid({:tag => 'a,b'}, false)
+    tracetag_valid({:tag => 'a?b'}, false)
+  end
+
+private
+
+  def tracetag_valid(attrs, result = true)
+    entry = Tracetag.new(gpx_file_tags(:first_trace_1).attributes, :without_protection => true)
+    entry.assign_attributes(attrs, :without_protection => true)
+    assert_equal result, entry.valid?, "Expected #{attrs.inspect} to be #{result}"
+  end
 end
-- 
2.39.5