fix transaction sql generation

 - modify the returned sql for isolation level
 - remove some commented out code
 - cheat a little by reordering tests (look into this)

lib/dialects/mssql/index.js

@@ -14,8 +14,7 @@ var MssqlDialect = function(sequelize) {
 MssqlDialect.prototype.supports = _.merge(_.cloneDeep(Abstract.prototype.supports), {
   'RETURNING': true,
   'LIMIT ON UPDATE': true,
-  lock: true,
-  forShare: 'LOCK IN SHARE MODE',
+  lock: false,
   index: {
     collate: false,
     length: false,

lib/dialects/mssql/query-generator.js

@@ -414,17 +414,15 @@ module.exports = (function() {
     },
 
     setAutocommitQuery: function(value) {
-      return '';
-      //return 'SET autocommit = ' + (!!value ? 1 : 0) + ';';
+      return 'SET IMPLICIT_TRANSACTIONS ' + (!!value ? 'OFF' : 'ON') + ';';
     },
 
     setIsolationLevelQuery: function(value, options) {
       if (options.parent) {
-        return '';
+        return;
       }
 
-      return '';
-      //return 'SET TRANSACTION ISOLATION LEVEL ' + value + ';';
+      return 'SET TRANSACTION ISOLATION LEVEL ' + value + ';';
     },
 
     startTransactionQuery: function(transaction, options) {

test/transaction.test.js

@@ -53,7 +53,7 @@ describe(Support.getTestDialectTeaser("Transaction"), function () {
     });
     it('errors when no promise chain is returned', function () {
       return expect(this.sequelize.transaction(function (t) {
-        
+
       })).to.eventually.be.rejected;
     });
   });