From e2b3a00100ebc52f29458d8fb24e817936ee16a7 Mon Sep 17 00:00:00 2001
From: marco <marco@5a81b35b-ba03-0410-adc8-b2c5c5119f08>
Date: Mon, 10 May 2010 05:39:01 +0000
Subject: [PATCH] fix alignment/sequence unit tests

git-svn-id: https://dng.biozentrum.unibas.ch/svn/openstructure/trunk@2219 5a81b35b-ba03-0410-adc8-b2c5c5119f08
---
 modules/seq/base/src/impl/sequence_impl.cc | 2 +-
 modules/seq/base/tests/test_alignment.cc   | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/modules/seq/base/src/impl/sequence_impl.cc b/modules/seq/base/src/impl/sequence_impl.cc
index c90e71c89..1572cd508 100644
--- a/modules/seq/base/src/impl/sequence_impl.cc
+++ b/modules/seq/base/src/impl/sequence_impl.cc
@@ -170,7 +170,7 @@ int SequenceImpl::GetLength() const {
 
 char SequenceImpl::GetOneLetterCode(int position) const
 {
-  if (position<0 || position>static_cast<int>(seq_string_.length()))
+  if (position<0 || position>=static_cast<int>(seq_string_.length()))
     throw Error("Position is not covered in sequence");
   return seq_string_[position];
 }
diff --git a/modules/seq/base/tests/test_alignment.cc b/modules/seq/base/tests/test_alignment.cc
index 97ffe338c..f8fef7fb6 100644
--- a/modules/seq/base/tests/test_alignment.cc
+++ b/modules/seq/base/tests/test_alignment.cc
@@ -215,9 +215,9 @@ BOOST_AUTO_TEST_CASE(alignment_aligned_region)
 
   BOOST_CHECK_THROW(ar = a.MakeRegion(-1,2,0),std::out_of_range);
   BOOST_CHECK_THROW(ar = a.MakeRegion(0,7,0),std::out_of_range);
-  BOOST_CHECK_THROW(ar = a.MakeRegion(2,4,0),std::out_of_range);
-  BOOST_CHECK_THROW(ar = a.MakeRegion(3,2,0),std::out_of_range);
-  BOOST_CHECK_THROW(ar = a.MakeRegion(3,3,0),std::out_of_range);
+  BOOST_CHECK_NO_THROW(ar = a.MakeRegion(2,4,0));
+  BOOST_CHECK_NO_THROW(ar = a.MakeRegion(3,2,0));
+  BOOST_CHECK_NO_THROW(ar = a.MakeRegion(3,3,0));
 }
 
 BOOST_AUTO_TEST_SUITE_END()
-- 
GitLab