Skip to content

Commit

Permalink
Stop looking for Config directory.
Browse files Browse the repository at this point in the history
The Config directory does not exist in NovaSeqX run folders.
  • Loading branch information
mgcam committed Mar 11, 2024
1 parent 8da8a47 commit 4424adf
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 8 deletions.
10 changes: 3 additions & 7 deletions lib/npg_tracking/illumina/run/folder.pm
Original file line number Diff line number Diff line change
Expand Up @@ -404,13 +404,9 @@ sub _get_path_from_given_path {
my @dirs = splitdir($subpath);
while (@dirs) {
my $path = catdir(@dirs);
if ( -d $path
and
-d catdir($path, $CONFIG_DIR) # does this directory have a Config Directory
and
-d catdir($path, $DATA_DIR) # a runfolder is likely to have a Data directory
) {
return $path;
# a runfolder has to have a Data directory
if (-d $path and -d catdir($path, $DATA_DIR)) {
return $path;
}
pop @dirs;
}
Expand Down
1 change: 0 additions & 1 deletion t/60-illumina-run-folder.t
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,6 @@ subtest 'standard runfolder, no DB access' => sub {
$paths->{no_archive_subpath} = $paths->{bbcalls_subpath} . q{/no_archive};
$paths->{pp_archive_subpath} = $paths->{bbcalls_subpath} . q{/pp_archive};
$paths->{qc_subpath} = $paths->{archive_subpath} . q{/qc};
$paths->{config_path} = $runfolder_path . q{/Config};

for my $path (values %{$paths}) {
make_path($path);
Expand Down

0 comments on commit 4424adf

Please sign in to comment.