Просмотр исходного кода

Remove latex-log-parser split test (#17999)

GitOrigin-RevId: 5accb0e5fc4c1ecce563f1d6f991de8c8633308b
Alf Eaton 2 лет назад
Родитель
Сommit
1f14746111

+ 0 - 3
services/web/app/src/Features/Project/ProjectController.js

@@ -553,9 +553,6 @@ const ProjectController = {
         pdfjsAssignment(cb) {
           SplitTestHandler.getAssignment(req, res, 'pdfjs-40', cb)
         },
-        latexLogParserAssignment(cb) {
-          SplitTestHandler.getAssignment(req, res, 'latex-log-parser', cb)
-        },
         trackPdfDownloadAssignment(cb) {
           SplitTestHandler.getAssignment(req, res, 'track-pdf-download', cb)
         },

+ 0 - 3
services/web/frontend/js/features/pdf-preview/util/output-files.js

@@ -1,4 +1,3 @@
-import getMeta from '../../../utils/meta'
 import HumanReadableLogs from '../../../ide/human-readable-logs/HumanReadableLogs'
 import BibLogParser from '../../../ide/log-parser/bib-log-parser'
 import { enablePdfCaching } from './pdf-caching-flags'
@@ -86,8 +85,6 @@ export const handleLogFiles = async (outputFiles, data, signal) => {
         result.log,
         {
           ignoreDuplicates: true,
-          oldRegexes:
-            getMeta('ol-splitTestVariants')?.['latex-log-parser'] !== 'new',
         }
       )
 

+ 3 - 14
services/web/frontend/js/ide/log-parser/latex-log-parser.js

@@ -9,10 +9,6 @@ const LINES_REGEX = /lines? ([0-9]+)/
 const PACKAGE_REGEX = /^(?:Package|Class|Module) (\b.+\b) Warning/
 const FILE_LINE_ERROR_REGEX = /^([./].*):(\d+): (.*)/
 
-const LATEX_WARNING_REGEX_OLD = /^LaTeX Warning: (.*)$/
-const PACKAGE_WARNING_REGEX_OLD = /^(Package \b.+\b Warning:.*)$/
-const PACKAGE_REGEX_OLD = /^Package (\b.+\b) Warning/
-
 const STATE = {
   NORMAL: 0,
   ERROR: 1,
@@ -27,16 +23,9 @@ export default class LatexParser {
     this.fileStack = []
     this.currentFileList = this.rootFileList = []
     this.openParens = 0
-    // TODO: Needed only for beta release; remove when over. 20220530
-    if (options.oldRegexes) {
-      this.latexWarningRegex = LATEX_WARNING_REGEX_OLD
-      this.packageWarningRegex = PACKAGE_WARNING_REGEX_OLD
-      this.packageRegex = PACKAGE_REGEX_OLD
-    } else {
-      this.latexWarningRegex = LATEX_WARNING_REGEX
-      this.packageWarningRegex = PACKAGE_WARNING_REGEX
-      this.packageRegex = PACKAGE_REGEX
-    }
+    this.latexWarningRegex = LATEX_WARNING_REGEX
+    this.packageWarningRegex = PACKAGE_WARNING_REGEX
+    this.packageRegex = PACKAGE_REGEX
     this.log = new LogText(text)
   }