Skip to content

Commit

Permalink
refactor(spm)!: Use the term SwiftPm in classes, files and package
Browse files Browse the repository at this point in the history
The Swift Package Manager folks do not use the term `Spm` for that
package manager. Align the name with upstream terminology, see also [1].

[1]: https://www.swift.org/package-manager/

Signed-off-by: Frank Viernau <frank_viernau@epam.com>
  • Loading branch information
fviernau committed Jan 22, 2024
1 parent 4116d16 commit 0289776
Show file tree
Hide file tree
Showing 6 changed files with 14 additions and 14 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
* License-Filename: LICENSE
*/

package org.ossreviewtoolkit.plugins.packagemanagers.spm
package org.ossreviewtoolkit.plugins.packagemanagers.swiftpm

import io.kotest.core.spec.style.WordSpec
import io.kotest.matchers.should
Expand All @@ -30,7 +30,7 @@ import org.ossreviewtoolkit.model.toYaml
import org.ossreviewtoolkit.utils.test.getAssetFile
import org.ossreviewtoolkit.utils.test.matchExpectedResult

class SpmFunTest : WordSpec({
class SwiftPmFunTest : WordSpec({
"Parsing 'Package.resolved' dependencies" should {
"return the correct result" {
val definitionFile = getAssetFile("projects/synthetic/spm-app/Package.resolved")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
* License-Filename: LICENSE
*/

package org.ossreviewtoolkit.plugins.packagemanagers.spm
package org.ossreviewtoolkit.plugins.packagemanagers.swiftpm

import java.io.File

Expand Down Expand Up @@ -50,22 +50,22 @@ private const val DEPENDENCIES_SCOPE_NAME = "dependencies"
/**
* The [Swift Package Manager](https://github.com/apple/swift-package-manager).
*/
class Spm(
class SwiftPm(
name: String,
analysisRoot: File,
analyzerConfig: AnalyzerConfiguration,
repoConfig: RepositoryConfiguration
) : PackageManager(name, analysisRoot, analyzerConfig, repoConfig), CommandLineTool {
private val graphBuilder = DependencyGraphBuilder(SpmDependencyHandler())
private val graphBuilder = DependencyGraphBuilder(SwiftPmDependencyHandler())

class Factory : AbstractPackageManagerFactory<Spm>(PACKAGE_TYPE) {
class Factory : AbstractPackageManagerFactory<SwiftPm>(PACKAGE_TYPE) {
override val globsForDefinitionFiles = listOf(PACKAGE_SWIFT_NAME, PACKAGE_RESOLVED_NAME)

override fun create(
analysisRoot: File,
analyzerConfig: AnalyzerConfiguration,
repoConfig: RepositoryConfiguration
) = Spm(type, analysisRoot, analyzerConfig, repoConfig)
) = SwiftPm(type, analysisRoot, analyzerConfig, repoConfig)
}

override fun command(workingDir: File?) = if (Os.isWindows) "swift.exe" else "swift"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,20 +17,20 @@
* License-Filename: LICENSE
*/

package org.ossreviewtoolkit.plugins.packagemanagers.spm
package org.ossreviewtoolkit.plugins.packagemanagers.swiftpm

import org.ossreviewtoolkit.model.Identifier
import org.ossreviewtoolkit.model.Issue
import org.ossreviewtoolkit.model.Package
import org.ossreviewtoolkit.model.PackageLinkage
import org.ossreviewtoolkit.model.PackageLinkage.DYNAMIC
import org.ossreviewtoolkit.model.utils.DependencyHandler
import org.ossreviewtoolkit.plugins.packagemanagers.spm.SwiftPackage.Dependency
import org.ossreviewtoolkit.plugins.packagemanagers.swiftpm.SwiftPackage.Dependency

/**
* A specialized [DependencyHandler] implementation for SPM.
*/
class SpmDependencyHandler : DependencyHandler<Dependency> {
class SwiftPmDependencyHandler : DependencyHandler<Dependency> {
override fun identifierFor(dependency: Dependency): Identifier = dependency.id

override fun dependenciesFor(dependency: Dependency): Collection<Dependency> = dependency.dependencies
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
* License-Filename: LICENSE
*/

package org.ossreviewtoolkit.plugins.packagemanagers.spm
package org.ossreviewtoolkit.plugins.packagemanagers.swiftpm

import kotlinx.serialization.SerialName
import kotlinx.serialization.Serializable
Expand Down
Original file line number Diff line number Diff line change
@@ -1 +1 @@
org.ossreviewtoolkit.plugins.packagemanagers.spm.Spm$Factory
org.ossreviewtoolkit.plugins.packagemanagers.swiftpm.SwiftPm$Factory
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,13 @@
* License-Filename: LICENSE
*/

package org.ossreviewtoolkit.plugins.packagemanagers.spm
package org.ossreviewtoolkit.plugins.packagemanagers.swiftpm

import io.kotest.core.spec.style.WordSpec
import io.kotest.inspectors.forAll
import io.kotest.matchers.shouldBe

class SpmTest : WordSpec({
class SwiftPmTest : WordSpec({
"getCanonicalName()" should {
"return the expected canonical name" {
listOf(
Expand Down

0 comments on commit 0289776

Please sign in to comment.