Tween.finished called if tween is stopped?

I hope Ashley can chime in here I don't wanna put it as a bugreport because it could be by design.

In Events:

If I start a tween (2seconds) that I then stop after 1 seconds. The tween just stops.

On Tween "Tag" finished does NOT trigger.

In Script:

class MySprite extends ISpriteInstance {
	constructor() {
		super();
	}

	async start() {
		this.tween = this.behaviors.Tween.startTween("x", this.x+200, 2, "linear");
		this.tween.finished.then(() => {
			this.setAnimation("Animation 2")
		})
	}

	cancel() {
		if(this.tween.stop) this.tween.stop();
	}
}

Ah pause... duh. Yeah that appears to work SORT OF with some adjustments.

There's just one thing that worries me though... I do in fact want the tween to end there, no need to resume. If I pause it, isn't it just floating around in memory indefinitely? And the async call is also kept waiting for the tween to end?

Some context, I'm using it for an attack animation. Here's how it looks in short

async attack() {
	if(this.attacking) return;
	this.attacking = tween;
	tween.finished.then(
		//do whatever
	)
}

cancelAttack() {
	this.attacking.pause();
	this.attacking = null;
}

WackyToaster Yeah if you do that, the reference to the tween will be kept around because technically it is never finished.

Right now I would keep track myself of what is going on and then do different things when the tween says it is finished.

Use a variable to keep track of when you stopped the tween. When the finished promise is resolved, check if you stopped it and do what you need to do and if you didn't stop it you can let it do something else.

async attack()
{
	if(this.attacking)
		return;
	
	this.attacking = tween;
	tween.finished.then(
		
		if (this.attacking)
		{
			// do something when finished with no interruption
		}
		else
		{
			// do something when cancelled
		}
	)
}

cancelAttack()
{
	this.attacking.stop();
	this.attacking = null;
}

Pretty sure your code does not work unless I mucked it up.

this.tween.finished will run before this.attacking = null. So the following check for if(this.attacking) will always be true.

But I do get the gist, I just need an extra variable to track.

I don't wanna implement it though if you plan on changing how this works anyway. :^) So if you plan on changing it I will post it to the tracker for tracking purposes. I'm really not acutely worried about a memory leak that happens if I spend an hour attacking.