diff --git a/test-suite/basismodels.cpp b/test-suite/basismodels.cpp
index 8ebc40cadc4..d160430bbab 100644
--- a/test-suite/basismodels.cpp
+++ b/test-suite/basismodels.cpp
@@ -131,6 +131,7 @@ namespace {
         std::vector<std::vector<Handle<Quote> > > capletVolQuotes;
         for (auto& capletVol : capletVols) {
             std::vector<Handle<Quote> > row;
+            row.reserve(capletVol.size());
             for (double j : capletVol)
                 row.push_back(RelinkableHandle<Quote>(ext::shared_ptr<Quote>(new SimpleQuote(j))));
             capletVolQuotes.push_back(row);
@@ -165,6 +166,7 @@ namespace {
         std::vector<std::vector<Handle<Quote> > > swaptionVolQuotes;
         for (auto& swaptionVol : swaptionVols) {
             std::vector<Handle<Quote> > row;
+            row.reserve(swaptionVol.size());
             for (double j : swaptionVol)
                 row.push_back(RelinkableHandle<Quote>(ext::shared_ptr<Quote>(new SimpleQuote(j))));
             swaptionVolQuotes.push_back(row);
diff --git a/test-suite/crosscurrencyratehelpers.cpp b/test-suite/crosscurrencyratehelpers.cpp
index 60b8cce13bf..07e2be13909 100644
--- a/test-suite/crosscurrencyratehelpers.cpp
+++ b/test-suite/crosscurrencyratehelpers.cpp
@@ -86,7 +86,7 @@ namespace crosscurrencyratehelpers_test {
                                              bool isBasisOnFxBaseCurrencyLeg) const {
             std::vector<ext::shared_ptr<RateHelper> > instruments;
             instruments.reserve(xccyData.size());
-            for (auto& i : xccyData) {
+            for (const auto& i : xccyData) {
                 instruments.push_back(constantNotionalXccyRateHelper(
                     i, collateralHandle, isFxBaseCurrencyCollateralCurrency,
                     isBasisOnFxBaseCurrencyLeg));