LDEV-5210 remove invalid test case

lucee · Dec 18, 2024 · 4e77853 · 4e77853
1 parent 3343d28
commit 4e77853
Showing 1 changed file with 0 additions and 30 deletions.
diff --git a/test/tickets/LDEV5206.cfc b/test/tickets/LDEV5206.cfc
@@ -86,20 +86,6 @@ component extends = "org.lucee.cfml.test.LuceeTestCase" {
 			});
 		});
 
-		describe( "LDEV-5210 DebugExecutionLog loop requires currentrow", function(){
-			xit( "LDEV-5210 test DebugExecutionLog - currentrow required ", function(){
-				var logs = getDebugLogs();
-				expect( len( logs ) ).toBe( 1 );
-				var log = logs[ 1 ];
-				expect( log ).toHaveKey( "pageParts" );
-				var pageParts = log.pageParts;
-				expect( pageParts ).toBeQuery();
-				//systemOutput( pageParts.toString(), true );
-				pageParts = _toPartsStructNoCurrentrow( pageParts );
-				expect( pageParts.row[1] ).toBe( 1 ); // returns ""
-				//systemOutput( pageParts.toString(), true );
-			});
-		});
 	}
 
 	private string function createURI( string calledName ){
@@ -134,29 +120,13 @@ component extends = "org.lucee.cfml.test.LuceeTestCase" {
 		var r = 0;
 		loop query=parts {
 			var r = parts.currentrow;
-			// TODO currentrow required LDEV-5210
 			querySetCell( parts, "path", mid( parts.path[ r ], len( dir ) ), r ); // less verbose
 			querySetCell( parts, "key", parts.path[ r ] & ":" & parts.startLine[ r ] & ":" & parts.endLine[ r ], r );
 		}
 		var st = QueryToStruct(parts, "key");
 		return st;
 	}
 
-	// currentrow required LDEV-5210
-	private function _toPartsStructNoCurrentrow( query pageParts ){
-		var parts = duplicate( pageParts );
-		queryAddColumn( parts, "row" );
-		var r = 0;
-		loop query=parts {
-			var r++;
-			querySetCell( parts, "row", r );
-		}
-		loop list="path,min,max,avg,count,total,id,start,end,snippet" item="local.col" {
-			queryDeleteColumn(parts, col);
-		}
-		return parts;
-	}
-
 	function getDebugLogs() cachedwithin="request" {
 		disableExecutionLog();
 		enableExecutionLog( "lucee.runtime.engine.DebugExecutionLog",{