with an empty array as a result of inferLanguage, we default on the latest supported version

Author: Janther

src/slang-utils/create-parser.ts

@@ -40,18 +40,28 @@ export function createParser(
 
     return result;
   }
+
   const inferredRanges: string[] = LanguageFacts.inferLanguageVersions(text);
   const inferredLength = inferredRanges.length;
 
-  if (inferredLength === 0) {
-    throw new Error(
-      `We couldn't infer a Solidity version base on the pragma statements in your code. If you would like to change that, update the pragmas in your source file, or specify a version in your \`.prettierrc\` file.`
+  if (inferredLength === 0 || inferredLength === supportedLength) {
+    const result = parserAndOutput(
+      text,
+      supportedVersions[supportedLength - 1]
     );
+
+    if (!result.parseOutput.isValid())
+      throw new Error(
+        `We encountered the following syntax error:\n\n\t${
+          result.parseOutput.errors()[0].message
+        }\n\nWe couldn't infer a Solidity version base on the pragma statements in your code so we defaulted to ${
+          result.parser.languageVersion
+        }. You might be attempting to use a syntax not yet supported by Slang or you might want to specify a version in your \`.prettierrc\` file.`
+      );
+    return result;
   }
-  const result = parserAndOutput(
-    text,
-    inferredRanges[inferredLength === supportedLength ? inferredLength - 1 : 0]
-  );
+
+  const result = parserAndOutput(text, inferredRanges[0]);
 
   if (!result.parseOutput.isValid())
     throw new Error(

tests/unit/slang-utils/create-parser.test.js

@@ -76,9 +76,7 @@ describe('inferLanguage', function () {
       description:
         'should use the latest version if the range is outside the supported versions',
       source: `pragma solidity ^10.0.0;`,
-      version: latestSupportedVersion,
-      // TODO: unskip this test when slack fixes the error with ranges outside the supported versions.
-      skip: true
+      version: latestSupportedVersion
     }
   ];
 
@@ -112,7 +110,7 @@ describe('inferLanguage', function () {
     expect(parser.languageVersion).toEqual('0.8.0');
   });
 
-  test('should throw an error if there are incompatible ranges', function () {
+  test.skip('should throw an error if there are incompatible ranges', function () {
     expect(() =>
       createParser(`pragma solidity ^0.8.0; pragma solidity 0.7.6;`, options)
     ).toThrow();