[lib] Validate DM operations
ClosedPublic
Actions

Authored by tomek on Sep 26 2024, 3:14 AM.

Details

Reviewers

kamil
will

Commits

rCOMM23ee267213b4: [lib] Validate DM operations

Summary

Add an explicit validation with some logs.

https://linear.app/comm/issue/ENG-9190/alternative-solution-update-canbeprocessed-functions

Depends on D13464

Test Plan

Clicked through the app and verified that e.g. sending text messages, reacting, and creating sidebars work.
Created some invalid operations from code and checked if it is discarded.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Sep 26 2024, 3:14 AM

Herald added a subscriber: ashoat. · View Herald TranscriptSep 26 2024, 3:14 AM

Harbormaster completed remote builds in B31873: Diff 44591.Sep 26 2024, 3:31 AM

tomek requested review of this revision.Sep 26 2024, 3:31 AM

tomek added a child revision: D13490: [lib] Handle a case where viewer is added to a thread while being its member.Sep 26 2024, 4:16 AM

kamil accepted this revision.Sep 26 2024, 7:35 AM

This revision is now accepted and ready to land.Sep 26 2024, 7:35 AM

Rebase

Harbormaster completed remote builds in B31897: Diff 44622.Sep 27 2024, 3:39 AM

Closed by commit rCOMM23ee267213b4: [lib] Validate DM operations (authored by tomek). · Explain WhySep 27 2024, 3:54 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM23ee267213b4: [lib] Validate DM operations.

Revision Contents
Changeset List

Path

Size

lib/

shared/

dm-ops/

add-members-spec.js

6 lines

add-viewer-to-thread-members-spec.js

6 lines

change-thread-read-status-spec.js

8 lines

change-thread-settings-spec.js

10 lines

change-thread-subscription.js

6 lines

create-entry-spec.js

6 lines

create-sidebar-spec.js

6 lines

create-thread-spec.js

8 lines

6 lines

3 lines

6 lines

6 lines

6 lines

5 lines

remove-members-spec.js

6 lines

send-edit-message-spec.js

6 lines

send-multimedia-message-spec.js

8 lines

send-reaction-message-spec.js

6 lines

send-text-message-spec.js

6 lines

update-relationship-spec.js

6 lines

Diff 44626

View Options

lib/shared/dm-ops/add-members-spec.js

Show All 35 Lines	public struct VideoInfo: Record {
var format: String = "N/A"		var format: String = "N/A"
}		}

public class MediaModule: Module {		public class MediaModule: Module {
public func definition() -> ModuleDefinition {		public func definition() -> ModuleDefinition {
Name("MediaModule")		Name("MediaModule")

AsyncFunction("getVideoInfo", getVideoInfo)		AsyncFunction("getVideoInfo", getVideoInfo)
		AsyncFunction("hasMultipleFrames", hasMultipleFrames)
}		}


// MARK: - Function implementations		// MARK: - Function implementations

private func getVideoInfo(path: URL) throws -> VideoInfo {		private func getVideoInfo(path: URL) throws -> VideoInfo {
let asset = AVAsset(url: path)		let asset = AVAsset(url: path)

Show All 15 Lines	private func getVideoInfo(path: URL) throws -> VideoInfo {
} else {		} else {
throw NoVideoTrackException(path)		throw NoVideoTrackException(path)
}		}

videoInfo.format = path.pathExtension		videoInfo.format = path.pathExtension

return videoInfo		return videoInfo
}		}

		private func hasMultipleFrames(path: URL) throws -> Bool {
		guard let imageSource = CGImageSourceCreateWithURL(path as CFURL, nil) else {
		throw FailedToLoadGifException(path)
		}

		let count = CGImageSourceGetCount(imageSource)

		return count > 1
		}
}		}

// MARK: - Exception definitions		// MARK: - Exception definitions

private class NoVideoTrackException: GenericException<URL> {		private class NoVideoTrackException: GenericException<URL> {
override var reason: String {		override var reason: String {
"No video track found in file URI: \(param)"		"No video track found in file URI: \(param)"
}		}
}		}

		private class FailedToLoadGifException: GenericException<URL> {
		override var reason: String {
		"Failed to load gif at URI: \(param)"
		}
		}

View Options

lib/shared/dm-ops/add-viewer-to-thread-members-spec.js

Show All 35 Lines	public struct VideoInfo: Record {
var format: String = "N/A"		var format: String = "N/A"
}		}

public class MediaModule: Module {		public class MediaModule: Module {
public func definition() -> ModuleDefinition {		public func definition() -> ModuleDefinition {
Name("MediaModule")		Name("MediaModule")

AsyncFunction("getVideoInfo", getVideoInfo)		AsyncFunction("getVideoInfo", getVideoInfo)
		AsyncFunction("hasMultipleFrames", hasMultipleFrames)
}		}


// MARK: - Function implementations		// MARK: - Function implementations

private func getVideoInfo(path: URL) throws -> VideoInfo {		private func getVideoInfo(path: URL) throws -> VideoInfo {
let asset = AVAsset(url: path)		let asset = AVAsset(url: path)

Show All 15 Lines	private func getVideoInfo(path: URL) throws -> VideoInfo {
} else {		} else {
throw NoVideoTrackException(path)		throw NoVideoTrackException(path)
}		}

videoInfo.format = path.pathExtension		videoInfo.format = path.pathExtension

return videoInfo		return videoInfo
}		}

		private func hasMultipleFrames(path: URL) throws -> Bool {
		guard let imageSource = CGImageSourceCreateWithURL(path as CFURL, nil) else {
		throw FailedToLoadGifException(path)
		}

		let count = CGImageSourceGetCount(imageSource)

		return count > 1
		}
}		}

// MARK: - Exception definitions		// MARK: - Exception definitions

private class NoVideoTrackException: GenericException<URL> {		private class NoVideoTrackException: GenericException<URL> {
override var reason: String {		override var reason: String {
"No video track found in file URI: \(param)"		"No video track found in file URI: \(param)"
}		}
}		}

		private class FailedToLoadGifException: GenericException<URL> {
		override var reason: String {
		"Failed to load gif at URI: \(param)"
		}
		}

View Options

lib/shared/dm-ops/change-thread-read-status-spec.js

Show All 35 Lines	public struct VideoInfo: Record {
var format: String = "N/A"		var format: String = "N/A"
}		}

public class MediaModule: Module {		public class MediaModule: Module {
public func definition() -> ModuleDefinition {		public func definition() -> ModuleDefinition {
Name("MediaModule")		Name("MediaModule")

AsyncFunction("getVideoInfo", getVideoInfo)		AsyncFunction("getVideoInfo", getVideoInfo)
		AsyncFunction("hasMultipleFrames", hasMultipleFrames)
}		}


// MARK: - Function implementations		// MARK: - Function implementations

private func getVideoInfo(path: URL) throws -> VideoInfo {		private func getVideoInfo(path: URL) throws -> VideoInfo {
let asset = AVAsset(url: path)		let asset = AVAsset(url: path)

Show All 15 Lines	private func getVideoInfo(path: URL) throws -> VideoInfo {
} else {		} else {
throw NoVideoTrackException(path)		throw NoVideoTrackException(path)
}		}

videoInfo.format = path.pathExtension		videoInfo.format = path.pathExtension

return videoInfo		return videoInfo
}		}

		private func hasMultipleFrames(path: URL) throws -> Bool {
		guard let imageSource = CGImageSourceCreateWithURL(path as CFURL, nil) else {
		throw FailedToLoadGifException(path)
		}

		let count = CGImageSourceGetCount(imageSource)

		return count > 1
		}
}		}

// MARK: - Exception definitions		// MARK: - Exception definitions

private class NoVideoTrackException: GenericException<URL> {		private class NoVideoTrackException: GenericException<URL> {
override var reason: String {		override var reason: String {
"No video track found in file URI: \(param)"		"No video track found in file URI: \(param)"
}		}
}		}

		private class FailedToLoadGifException: GenericException<URL> {
		override var reason: String {
		"Failed to load gif at URI: \(param)"
		}
		}

View Options

lib/shared/dm-ops/change-thread-settings-spec.js

Show All 35 Lines	public struct VideoInfo: Record {
var format: String = "N/A"		var format: String = "N/A"
}		}

public class MediaModule: Module {		public class MediaModule: Module {
public func definition() -> ModuleDefinition {		public func definition() -> ModuleDefinition {
Name("MediaModule")		Name("MediaModule")

AsyncFunction("getVideoInfo", getVideoInfo)		AsyncFunction("getVideoInfo", getVideoInfo)
		AsyncFunction("hasMultipleFrames", hasMultipleFrames)
}		}


// MARK: - Function implementations		// MARK: - Function implementations

private func getVideoInfo(path: URL) throws -> VideoInfo {		private func getVideoInfo(path: URL) throws -> VideoInfo {
let asset = AVAsset(url: path)		let asset = AVAsset(url: path)

Show All 15 Lines	private func getVideoInfo(path: URL) throws -> VideoInfo {
} else {		} else {
throw NoVideoTrackException(path)		throw NoVideoTrackException(path)
}		}

videoInfo.format = path.pathExtension		videoInfo.format = path.pathExtension

return videoInfo		return videoInfo
}		}

		private func hasMultipleFrames(path: URL) throws -> Bool {
		guard let imageSource = CGImageSourceCreateWithURL(path as CFURL, nil) else {
		throw FailedToLoadGifException(path)
		}

		let count = CGImageSourceGetCount(imageSource)

		return count > 1
		}
}		}

// MARK: - Exception definitions		// MARK: - Exception definitions

private class NoVideoTrackException: GenericException<URL> {		private class NoVideoTrackException: GenericException<URL> {
override var reason: String {		override var reason: String {
"No video track found in file URI: \(param)"		"No video track found in file URI: \(param)"
}		}
}		}

		private class FailedToLoadGifException: GenericException<URL> {
		override var reason: String {
		"Failed to load gif at URI: \(param)"
		}
		}